New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate from deprecated cli-ux to @oclif/core #969
Conversation
This is great! We'll try to get this tested when we have some free time. Do you think you'll have the spare time to migrate the old packages to |
Yep, sure. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you!
Summary
cli-ux package is deprecated now. This PR contains migration from cli-ux to @oclif/core.
All changes were made by instructions from package migration manual.
Also, there are some other "old" @oclif/* packages in project (like @oclif/command, @oclif/config, etc). They all marked as "in maintenance mode" and replaced by @oclif/core. It's probably better to do their migration in another PR.
Testing Plan
Whole refactoring pretty transparent: import another namespace from new package and use it instead of old one. I tested every refactored command manually and ran
yarn test
for sure. All tests passed.Breaking Change
Is this a breaking change? If yes, add notes below on why this is breaking and
what additional work is required, if any.