Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds CSS Linter and Formatter #116

Closed

Conversation

mechamobau
Copy link

@mechamobau mechamobau commented Jul 22, 2021

In this PR is added a CSS Linter and Formatter to the project and created a step to verify the CSS on new Pull Requests.

I've added the Node.js environment to the action because I have no familiarity with Nix environments, so I'm open to change that according to any given suggestions.

To run that locally it's needed to have Node.js installed and run the following command inside the root directory:

$ npx stylelint 'css/*.css' --fix

closes #110

@TikhonJelvis
Copy link
Member

Thanks for the config. It would be great to do this through Nix rather than installing NPM separately.

I might give it a try myself, but don't have the time/attention at the moment. In the meantime, please feel free to give it a go yourself if you're interested in learning!

@TikhonJelvis
Copy link
Member

Looks like this will need some Nix work to be useful, and I probably won't get around to it any time soon. Going to close the PR for now, and we can always pick it up again later if we want linting/minimization for our CSS in the future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add a CSS Auto-Formatter
2 participants