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
This ticket addresses eslint warnings for rules copied from the v0.3 branch.
Instructions
Review the rule documentation for the rule (see reference pages below)
Navigate the cli to the client folder
Run the lint script (e.g. npm run lint)
Review the results, noting all instances where the rule is referenced.
Address each warning using one of the following options, according to your best judgement:
use the npx eslint --fix --rule {rule to fix} command (not available for all rules)
correct the code according to the recommendations listed in the rule documentation
disable the rule for the reported line (NOTE: only use the disable instruction for a single line. Do NOT use the more global options). Include a comment for why this decision was made.
raise the rule as a topic to discuss with the group in the next meeting. We can then decide whether to edit or drop the rule, add more global disable flags, etc.
Acceptance Criteria
Running the lint script should result in an output that does not mention this rule (meaning all instances either corrected or marked for ignore).
v0.4 Setup Infrastructure
This ticket addresses eslint warnings for rules copied from the v0.3 branch.
Instructions
client
foldernpm run lint
)npx eslint --fix --rule {rule to fix}
command (not available for all rules)Acceptance Criteria
Running the lint script should result in an output that does not mention this rule (meaning all instances either corrected or marked for ignore).
Reference
The text was updated successfully, but these errors were encountered: