-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
docs: adopt eslint-stylistic sub packages related changes #19887
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
Conversation
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.
Pull Request Overview
This PR updates package references from the old @stylistic/js
scope to the new @stylistic
scope, adjusting plugin names, URLs, and rule links across tests, rule metadata, and documentation.
- Update deprecated rule replacement references in tests.
- Update
plugin.name
andplugin.url
in all rule metadata to@stylistic/eslint-plugin
andhttps://eslint.style
. - Remove
/js/
segment from all rule URLs in metadata and documentation.
Reviewed Changes
Copilot reviewed 78 out of 80 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
tests/lib/eslint/eslint.js | Updated replacedBy entries to new package names |
lib/rules/*.js | Updated plugin name, URL, and rule URLs |
docs/src/use/formatters/index.md | Updated example formatter output URLs |
docs/src/_data/rules_meta.json | Updated plugin info and rule URLs |
docs/src/_data/rules.json | Updated plugin info and rule URLs |
✅ Deploy Preview for docs-eslint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
Thanks for your work on this PR.
I’ve left a minor comment regarding the EOL in the autogenerated file.
To the team: I’ve manually checked all the links, and they’re working as expected. 👍
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, Thanks!
Would like another review before merging.
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, thanks.
Marking this change as docs
because there is no direct user impact.
Prerequisites checklist
What is the purpose of this pull request? (put an "X" next to an item)
[x] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[x] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:
Fixes: #19820
What changes did you make? (Give an overview)
This PR updates all eslint-stylistic sub packages related links and plugin names.
Is there anything you'd like reviewers to focus on?