-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Closed
Description
Nature of issue?
- New feature request (This would likely reuse Enhancement : Add a .prettierrc file to match with the .eslintrc #861 )
New feature details:
- Setting up https://github.com/marketplace/actions/lint-action would give developers feedback immediately if their change introduces a regression in formatting. This would prevent the need for commits like https://github.com/Kaushal-Dhungel/p5.js-web-editor/commit/d444aca7a2627d4f4f741d5478ab416adcfe438a .
- The integration has the ability to auto-fix errors. However, I could see this possibly being confusing to new contributors if they don't know about how to merge those changes back into their local branch. I'd recommend erring on the side of using client-side pre-commit hooks instead if we wanted to make auto-fixes part of the developer experience. https://prettier.io/docs/en/precommit.html
Metadata
Metadata
Assignees
Labels
No labels