-
Notifications
You must be signed in to change notification settings - Fork 36
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
Document Prettier in README #2104
Conversation
…esomeImports.js to .prettierignore, and create an npm command to simplify formatting checks
✅ Deploy Preview for docs-spectrocloud ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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 great, just want to circle back about the commands we are sharing.
README.md
Outdated
In this case, issue the following command to format all files manually. | ||
|
||
``` | ||
npm run format |
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.
Can we use the Makefile instead? That's a pattern we started a long time ago.
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've added the Make command. Could you please document this pattern so that it's explicit for anyone who wants to introduce changes to the docbase? I would also be happy to document it if you have a minute to explain the patterns expectations to me 🙌🏻
README.md
Outdated
commit changes, Prettier formats the staged files automatically. Then, once you create a pull request, it verifies that | ||
all files comply with our Prettier configuration. | ||
|
||
**Note:** The build fails if the Code Formatting check doesn't pass. |
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.
Because this is for the README. We could use GitHub markdown alerts.
* chore: Document Prettier, add src/components/IconMapper/dynamicFontAwesomeImports.js to .prettierignore, and create an npm command to simplify formatting checks * chore: Update Prettier docs and add format-check to Makefile * chore: Add docs/api-content/**/*.json to .prettierignore --------- Co-authored-by: Karl Cardenas <karl@spectrocloud.com> (cherry picked from commit 5b03006)
* chore: Document Prettier, add src/components/IconMapper/dynamicFontAwesomeImports.js to .prettierignore, and create an npm command to simplify formatting checks * chore: Update Prettier docs and add format-check to Makefile * chore: Add docs/api-content/**/*.json to .prettierignore --------- Co-authored-by: Karl Cardenas <karl@spectrocloud.com> (cherry picked from commit 5b03006)
* chore: Document Prettier, add src/components/IconMapper/dynamicFontAwesomeImports.js to .prettierignore, and create an npm command to simplify formatting checks * chore: Update Prettier docs and add format-check to Makefile * chore: Add docs/api-content/**/*.json to .prettierignore --------- Co-authored-by: Karl Cardenas <karl@spectrocloud.com> (cherry picked from commit 5b03006)
💔 Some backports could not be created
Note: Successful backport PRs will be merged automatically after passing CI. Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation and see the Github Action logs for details |
* chore: Document Prettier, add src/components/IconMapper/dynamicFontAwesomeImports.js to .prettierignore, and create an npm command to simplify formatting checks * chore: Update Prettier docs and add format-check to Makefile * chore: Add docs/api-content/**/*.json to .prettierignore --------- Co-authored-by: Karl Cardenas <karl@spectrocloud.com> (cherry picked from commit 5b03006) Co-authored-by: Yuliia Horbenko <31223054+yuliiiah@users.noreply.github.com>
* chore: Document Prettier, add src/components/IconMapper/dynamicFontAwesomeImports.js to .prettierignore, and create an npm command to simplify formatting checks * chore: Update Prettier docs and add format-check to Makefile * chore: Add docs/api-content/**/*.json to .prettierignore --------- Co-authored-by: Karl Cardenas <karl@spectrocloud.com> (cherry picked from commit 5b03006) Co-authored-by: Yuliia Horbenko <31223054+yuliiiah@users.noreply.github.com>
* chore: Document Prettier, add src/components/IconMapper/dynamicFontAwesomeImports.js to .prettierignore, and create an npm command to simplify formatting checks * chore: Update Prettier docs and add format-check to Makefile * chore: Add docs/api-content/**/*.json to .prettierignore --------- Co-authored-by: Karl Cardenas <karl@spectrocloud.com> (cherry picked from commit 5b03006) Co-authored-by: Yuliia Horbenko <31223054+yuliiiah@users.noreply.github.com>
🎉 This PR is included in version 4.2.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 4.2.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Describe the Change
This PR:
npm run format-check
.Review Changes
💻 Add Preview URL
🎫 Jira Ticket