Skip to content

chore: remove @oclif/plugin-legacy dependency#3659

Merged
eablack merged 3 commits intomainfrom
eb/remove-plugin-legacy
Apr 14, 2026
Merged

chore: remove @oclif/plugin-legacy dependency#3659
eablack merged 3 commits intomainfrom
eb/remove-plugin-legacy

Conversation

@eablack
Copy link
Copy Markdown
Contributor

@eablack eablack commented Apr 14, 2026

Summary

This PR removes the @oclif/plugin-legacy dependency from the project, as it is no longer needed. The plugin has been removed from both package.json dependencies and the oclif plugins configuration.

Type of Change

Patch Updates (patch semver update)

  • chore: Change that does not affect production code

Testing

Notes:
Verify that the CLI still builds and runs correctly without the legacy plugin.

Steps:

  1. Run npm install to verify package-lock.json is valid
  2. Run npm run build to ensure the build succeeds
  3. Run basic CLI commands to verify functionality
  4. Passing CI suffices

This removes the @oclif/plugin-legacy plugin which is no longer needed.
@eablack eablack requested a review from a team as a code owner April 14, 2026 00:23
This type definition was previously provided transitively through
@oclif/plugin-legacy. Now that the legacy plugin is removed, we need
to explicitly include it as a devDependency.
@eablack eablack temporarily deployed to AcceptanceTests April 14, 2026 00:49 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests April 14, 2026 00:49 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests April 14, 2026 00:49 — with GitHub Actions Inactive
@eablack eablack temporarily deployed to AcceptanceTests April 14, 2026 00:49 — with GitHub Actions Inactive
Copy link
Copy Markdown
Contributor

@sbosio sbosio left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@tlowrimore-heroku tlowrimore-heroku left a comment

Choose a reason for hiding this comment

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

LGTM!

@eablack eablack merged commit 2523d48 into main Apr 14, 2026
17 checks passed
@eablack eablack deleted the eb/remove-plugin-legacy branch April 14, 2026 17:07
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.

3 participants