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
Move core modules to @textlint namespace #455
Comments
@azu, I have a question; do we migrate Since |
@0x6b Good question! I've updated migration name. |
@azu, thank you for the clarification. It's good move.
+1 for |
I decided that rename |
@azu, thank you. I'm on my way to migrate
So, while migrating, I think I need two pull requests. Am I missing anything?
|
Yes! I've tried that move It is better that complete migration task in a single PR :) I've added @0x6b to npm @textlint member. FYI: |
@azu, thank you for adding me to the npm organization! |
@azu, while refactoring
|
Probably, you need to change plugin name.
- const engine = new TextLintEngine({ plugins: ["markdown"] });
+ const engine = new TextLintEngine({ plugins: ["@textlint/markdown"] }); |
@azu, you are right! (as always)
|
$ for package in txt-to-ast markdown-to-ast textlint-ast-test txt-ast-traverse textlint-fixer-formatter textlint-formatter textlint-plugin-markdown textlint-plugin-text; do echo -n "$package: "; npm info --json $package | jq .deprecated; done
txt-to-ast: "See https://github.com/textlint/textlint/issues/455"
markdown-to-ast: "See https://github.com/textlint/textlint/issues/455"
textlint-ast-test: "See https://github.com/textlint/textlint/issues/455"
txt-ast-traverse: "See https://github.com/textlint/textlint/issues/455"
textlint-fixer-formatter: "See https://github.com/textlint/textlint/issues/455"
textlint-formatter: "See https://github.com/textlint/textlint/issues/455"
textlint-plugin-markdown: "See https://github.com/textlint/textlint/issues/455"
textlint-plugin-text: "See https://github.com/textlint/textlint/issues/455" Now we can go to another set of refactoring such as TypeScript-ify or Jest-ing (#451). |
Found that $ for package in txt-to-ast markdown-to-ast textlint-ast-test txt-ast-traverse textlint-fixer-formatter textlint-formatter textlint-plugin-markdown textlint-plugin-text; do echo -n "$package: "; npm view $package deprecated; done
txt-to-ast: See https://github.com/textlint/textlint/issues/455
markdown-to-ast: See https://github.com/textlint/textlint/issues/455
textlint-ast-test: See https://github.com/textlint/textlint/issues/455
txt-ast-traverse: See https://github.com/textlint/textlint/issues/455
textlint-fixer-formatter: See https://github.com/textlint/textlint/issues/455
textlint-formatter: See https://github.com/textlint/textlint/issues/455
textlint-plugin-markdown: See https://github.com/textlint/textlint/issues/455
textlint-plugin-text: See https://github.com/textlint/textlint/issues/455 |
* chore: replace textlint-ast-test with @textlint/ast-tester refs textlint/textlint#455 * fix: run 'npm audit fix' command
ℹ️ Migration information
For textlint user
If you have seen following deprecated warning, please update lateste
textlint
,textlint-tester
, ortextlint-scripts
.Update latest version
For node modules user
If you use someone of Migration pacakges, please move to scoped version.
For example, If you use
markdown-to-ast
, you migrate frommarkdown-to-ast
to@textlint/markdown-to-ast
.And, you should change
require("markdown-to-ast")
torequire("@textlint/markdown-to-ast")
.npm recommened to use scoped.
We already have @textlint scope.
Pros
For example, markdown-to-ast is confused name.
Cons
But, almost pacakges are only used in textlint.
Migration pacakges
txt-to-ast
@textlint/text-to-ast
markdown-to-ast
@textlint/markdown-to-ast
textlint-ast-test
@textlint/ast-tester
txt-ast-traverse
@textlint/ast-traverse
textlint-fixer-formatter
@textlint/fixer-formatter
textlint-formatter
@textlint/linter-formatter
textlint-plugin-markdown
@textlint/textlint-plugin-markdown
textlint-plugin-text
@textlint/textlint-plugin-text
Migration Packages List
@textlint/text-to-ast
refactor(text-to-ast): Move txt-to-ast to @textlint/text-to-ast #473@textlint/markdown-to-ast
refactor(markdown-to-ast): move markdown-to-ast to @textlint scope #460@textlint/ast-tester
refactor(textlint-ast-test): move to @textlint/ast-tester #483er
is good@textlint/ast-traverse
refactor(txt-ast-traverse): move to @textlint/ast-traverse #484@textlint/fixer-formatter
refactor(textlint-fixer-formatter): move to @textlint/fixer-formatter #482@textlint/linter-formatter
refactor(textlint-formatter): move to @textlint/linter-formatter #474@textlint/textlint-plugin-markdown
refactor(plugin-markdown): move to @textlint #485@textlint/textlint-plugin-text
refactor(plugin-text): move to @textlint #486textlint-plugin
prefix is neededNo migration packages
These packages are used by external user.
How to migrate?
Migrate to
@textlint
.Maybe, transfer-github-owner help this.Example: #473 #460
<name>
to@textlint/<name>
directory 710a09e<name>
to@textlint/<name>
5a12178yarn.lock
byyarn bootstrap
"publishConfig": { "access": "public" }
config is needed when you publish Scoped packages at first timenpm publish --access public
is anthor optionpublishConfig
is more reasonable.npm publish
it as@textlint/<name>
from localFinally, deprecate old pacakge:
The text was updated successfully, but these errors were encountered: