-
Notifications
You must be signed in to change notification settings - Fork 109
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
docs: Rework naming convention and status description requirements #118
Conversation
…for better sorting
I can add a lint feature later. |
You're welcome :) @hahayupgit any thoughts on PR? |
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.
Overall looks good! Minor things I'm going to go in and fix, and then approve. Great job!!!!
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.
Fantastic PR!
Description
Type(s)
Verification
Have you
./scripts/generate.mjs
?./scripts/lint.mjs
?Most likely the biggest PR of all times here but it addresses some things that make docs navigation, maintenance and lack of some details issues:
[!WARNING]
heading to keep any games issues in one place.REDlauncher
in The Witcher 3.Any maintainer feedback is welcome as it is a huge change and it is better to discuss it before deploying but I hope that my idea and implementation have a solid background.