You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Since proper linting and formatting checks were added in #193, all of them have always failed. We should start addressing this. From build previews, the key issue we have is code formatting and linting (formatting is an easy fix but the latter requires a little bit of work to the codebase because it's manual).
We have checks in place for both in package.json, and we need to start enforcing these standards before approving PRs. So before someone's PR is approved, they should, among other things, run the prettier:format and lint scripts in package.json (added in #193).
That way, one gets a chance to fix these before blindly making a PR. We should also update the PR template to let everyone know while making a PR.
Tasks:
Fix linting issues
Fix formatting issues
Update PR template
The text was updated successfully, but these errors were encountered:
Since proper linting and formatting checks were added in #193, all of them have always failed. We should start addressing this. From build previews, the key issue we have is code formatting and linting (formatting is an easy fix but the latter requires a little bit of work to the codebase because it's manual).
We have checks in place for both in
package.json
, and we need to start enforcing these standards before approving PRs. So before someone's PR is approved, they should, among other things, run theprettier:format
andlint
scripts inpackage.json
(added in #193).That way, one gets a chance to fix these before blindly making a PR. We should also update the PR template to let everyone know while making a PR.
Tasks:
The text was updated successfully, but these errors were encountered: