Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.8k
chore(website): generate rule docs h1 and description automatically#5249
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
chore(website): generate rule docs h1 and description automatically#5249
Uh oh!
There was an error while loading.Please reload this page.
Conversation
nx-cloudbot commentedJun 25, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Thanks for the PR,@JoshuaKGoldberg! typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community. The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately. Thanks again! 🙏Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently onhttps://opencollective.com/typescript-eslint. As a thank you, your profile/company logo will be added to our main README which receives thousands of unique visitorsper day. |
netlifybot commentedJun 25, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
✅ Deploy Preview fortypescript-eslint ready!
To edit notification comments on pull requests, go to yourNetlify site settings. |
}, | ||
], | ||
type: 'blockquote', | ||
} as mdast.Blockquote); |
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.
Docusaurus adds anh1
automatically if the file doesn't already start with one. So all we need to add is the description.
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.
- Maybe use a paragraph instead of a quote?
- Add this to the
RuleAttributes
component instead of in Markdown? Thedescription
is already part of the rule metadata available on client side.
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'm not sure about this one, docs at this point is slowly starting unusable in github |
I think the ideais that the docs should not be viewed on GitHub but should be as maintainable as possible. |
if that is a case than we should move it out of eslint-plugin to docs |
Yeah, this brings up a good point - it seems like#4980 is going to be particularly relevant now that the |
@@ -1,7 +1,3 @@ | |||
# `your-rule-name` |
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.
@armano2 starting a thread for#5249 (comment):
I think the idea is that the docs should not be viewed on GitHub but should be as maintainable as possible.
if that is a case than we should move it out of eslint-plugin to docs
I think I agree, it would be best long term to have these docs files in the docs area. But: the old .md files still have much better SEO than the site. We can't move them all-up just yet. The most we could do would be to leave a .md file in the old location with just the redirect notice.
What would you prefer I do in this PR? I don't have strong feelings one way or another, as long as we're moving towards automating more of the rule docs pages.
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.
Personally - I think it's still a good idea to co-locate them with the plugin - that way they're easy to find and update for contributors.
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 am 100% behind treating the .md files in the repo as source code for most things (i.e. users shouldn't read them).
It's what I said a year ago with the first versions of the website!
Also LMAO I had a different opinion back then about where to put the files:
TBH - I also don't see any tangible value in co-locating the docs with the packages.
So I'll say - I don't mind either way. As long as we keep things clearly documented for contributors (i.e. tests to fail if they've not been added, or not been deleted) - then it shouldn't matter.
Uh oh!
There was an error while loading.Please reload this page.
PR Checklist
Overview
Continues the general work started in#5085 around automating the base template for rule docs. This time the
#
h1 and next description are added automatically. I changed the format a bit to match the non-code h1 and slightly different description from ESLint rule docs: e.g.https://eslint.org/docs/latest/rules/array-callback-return.Followups that could be filed as issues:
## Rule Details