@burtonator wrote a cheatsheet for handling common linting issues.
I've integrated that cheatsheet with our current knowledge base entry, Linting.md. Some (relatively minor) changes were made to the cheatsheet's contents, either to (1) remove redundancies with the existing documentation, (2) reorganize information from the cheatsheet into another, more relevant section of the entry, or (3) bring it in line stylistically with the knowledge base. Nothing major, but just flagging this explicitly @burtonator so you're aware, and scan over it accordingly!
I've also reviewed and refreshed the existing documentation on linting and formatting to the best of my knowledge—however, I don't use these tools day in and day out, so I'd welcome eyes from @masvelio, @kurtisassad or any other team member. (Picking on you two since you've expressed opinions in the past...)
Link to Issue
Closes: #7537
Description of Changes
Reorganizes the entry into two main sections, one on eslint and one on prettier, plus the cheatsheet
A few lines acknowledging the recent introduction of eslint-diff plugin—@burtonator, if the inclination strikes you, feel free to suggest more language/info here
"Deployment Plan"
Will mark as "Certified Fresh" in the change log once I get some approvals / feedback.
@burtonator wrote a cheatsheet for handling common linting issues.
I've integrated that cheatsheet with our current knowledge base entry,
Linting.md. Some (relatively minor) changes were made to the cheatsheet's contents, either to (1) remove redundancies with the existing documentation, (2) reorganize information from the cheatsheet into another, more relevant section of the entry, or (3) bring it in line stylistically with the knowledge base. Nothing major, but just flagging this explicitly @burtonator so you're aware, and scan over it accordingly!I've also reviewed and refreshed the existing documentation on linting and formatting to the best of my knowledge—however, I don't use these tools day in and day out, so I'd welcome eyes from @masvelio, @kurtisassad or any other team member. (Picking on you two since you've expressed opinions in the past...)
Link to Issue
Closes: #7537
Description of Changes
eslint-diffplugin—@burtonator, if the inclination strikes you, feel free to suggest more language/info here"Deployment Plan"
Will mark as "Certified Fresh" in the change log once I get some approvals / feedback.