search

cdnjs / new-website

🖥 cdnjs.com website
https://cdnjs.com
ISC License
451 stars 212 forks source link

public/css/*.css needs to be linted #299

Closed MattIPv4 closed 4 years ago

MattIPv4 commented 5 years ago

We are adding linting for all main files relating to the website. We have already implemented eslint for the backend scripts and linting for the HTML templates and front-end JS are in the works.

We should also be linting the main css files that are used for the site to ensure they also keep a good code style standard going forward.

drewfreyling commented 5 years ago

What are you thinking? Stylelint?

MattIPv4 commented 5 years ago

Hey @drewfreyling, yup, I think I plan to go for stylelint over csslint. Do you have any preference on what base rules we use, or should I just build up a config based around what we have already?

drewfreyling commented 5 years ago

@MattIPv4 I normally just use stylelint-config-standard rules