-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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.nameandplugin.urlin all rule metadata to@stylistic/eslint-pluginandhttps://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. |
lumirlumir
left a comment
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. 👍
lumirlumir
left a comment
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.
snitin315
left a comment
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?