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
refactor(textlint-formatter): move to @textlint/linter-formatter #474
Conversation
No. Runtime message break exiting behavior. Instead of it, You can deprecate old module by executing following command:
As as result, show deprecated message in installing time.
|
Yes and No.
|
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.
LGTM.
Please do npm depreacate
old module before merging.
#474 (comment)
For the record:
|
Ref #455.
Note: I did not change CLI name from
textlint-formatter
tolinter-formatter
for compatibility. I thinktextlint-formatter
is more intuitive name for users, but if you thinklinter-formatter
is better, please let me know and I will update this pull request as follows:linter-formatter.js
tobin/
directorypackage.json#bin
as keylinter-formatter
textlint-formatter
(bin/cmd.js
)Need major version bump as it will break CI build. Thanks.