Skip to content

refactor: clean up project configuration and remove deprecated code#36

Open
DRiFTy17 wants to merge 3 commits intomainfrom
legacy-command-removal
Open

refactor: clean up project configuration and remove deprecated code#36
DRiFTy17 wants to merge 3 commits intomainfrom
legacy-command-removal

Conversation

@DRiFTy17
Copy link
Collaborator

@DRiFTy17 DRiFTy17 commented Jul 22, 2025

  • Removed legacy lint and test commands. These are to be handled by individual projects going forward.
  • Updated dependencies
  • Switches from Auto to Changesets CLI for release managements
  • Updated tsconfig settings
  • Removed unused code

@DRiFTy17 DRiFTy17 requested a review from a team as a code owner July 22, 2025 20:46
@changeset-bot
Copy link

changeset-bot bot commented Jul 22, 2025

🦋 Changeset detected

Latest commit: 3409a93

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@tylertech/forge-cli Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@DRiFTy17 DRiFTy17 requested a review from Copilot July 22, 2025 20:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the project by removing deprecated test and lint commands, cleaning up unused code, switching from Auto to Changesets for release management, and modernizing the build toolchain.

Key changes include:

  • Removed legacy test and lint commands and their dependencies
  • Switched from Auto to Changesets CLI for release management
  • Updated TypeScript configuration and ESLint setup
  • Added Prettier for code formatting

Reviewed Changes

Copilot reviewed 36 out of 39 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/utils/worker-pool.ts Completely removed worker pool implementation
src/utils/utils.ts Removed utility functions and added error handling helpers
src/utils/test-utils.ts Completely removed test utilities
src/utils/lint-utils.ts Completely removed lint utilities
src/utils/build-utils.ts Cleaned up imports and updated TypeScript configuration
src/core/definitions.ts Removed deprecated interfaces and test-related configurations
src/core/configuration.ts Removed OS detection and simplified constructor
src/core/cli.ts Removed test and lint commands from available commands
package.json Updated dependencies and scripts, removed test/lint related packages
eslint.config.js New ESLint configuration file
.changeset/* Added Changesets configuration files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants