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
feat: Simplified CS creation flow ALA-940 #2074
base: master
Are you sure you want to change the base?
feat: Simplified CS creation flow ALA-940 #2074
Conversation
Overall readability score: 53.82 (🔴 -0.01)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
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.
Looks very good, @nicklem! 👌
I like the way you reorganized the content to fit the advanced settings. 👍
I left some comments for your evaluation.
- To toggle multiple code patterns at once, click the checkbox of the first pattern and **Shift+click** the checkbox of the last pattern in a range. | ||
- To toggle all the code patterns visible on the list, click the checkbox on the header of the code patterns list. If there are more code patterns to load on the list, you can click the link **Enable/Disable all <N> patterns** to toggle all patterns matching the current filters. | ||
|
||
![Configuring the tools and patterns for the coding standard](images/coding-standard-configure-tools.png)<1-- TODO ALA-940 screenshot --> | ||
|
||
## Defining a default coding standard {: id="set-default"} |
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.
We could mention (maybe here) the option to set a coding standard as default during when creating the 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.
Sorry, the link doesn't work after I pushed some updates. Where did you suggest that we mention 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.
Somewhere near this line:
To set a coding standard as default:
(GitHub didn't allow me to add that note on the specific line because it wasn't new content... 😉 )
Co-authored-by: Cláudia Carpinteiro <claudiacarpinteiro@users.noreply.github.com>
❗ WIP
Documents the new, simplified coding standard creation flow.
👀 Live preview
https://ala-940-docs-document-simplified-cs--docs-codacy.netlify.app/organizations/using-coding-standards/
🚧 To do