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.
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: