Update: remove meta.docs.category in core rules #848
Conversation
✔️ Deploy Preview for eslint ready! 🔨 Explore the source changes: b6207d5 🔍 Inspect the deploy log: https://app.netlify.com/sites/eslint/deploys/6123d041c70f2000079a6ce3 😎 Browse the preview: https://deploy-preview-848--eslint.netlify.app/docs/rules |
Can you rebase this? |
0cf59f1
to
9d0d609
Compare
I think we need to have headings that look a bit nicer: problem -> Possible Problems |
it was auto-gened. I just added a new field "displayName"(the name was borrowed from react:)). eslint/eslint@ef4c31b |
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.
I like it. 👍
👍 good call! will update the PR in eslint repo. |
90f6ffc
to
9dc1253
Compare
Just a reminder to use a PR draft if you don’t want a PR to be merged. |
@aladdin-add eslint/eslint#14594 has multiple approvals and eslint/eslint#13398 has been moved to the Ready for Merge state in the v8.0.0 project. You can now undo the |
9dc1253
to
4674cf4
Compare
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!
Note to the merger: This PR is ready for merge as-is, but merging is blocked on the ESLint v8.0.0 final release. ESLint v7 releases produce Therefore:
|
@aladdin-add can you resolve merge conflicts? |
I addressed the conflict (my change originally caused it). |
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!
(#848 (comment) still applies, do not merge this before v8.0.0 final release).
Merging this now that |
refs: eslint/eslint#13398