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.

Sign up for GitHub

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

Jump to bottom

feature: add formatting and linting #148

Merged

supersven merged 1 commit into alpmestan:master from MangoIV:mangoiv/linting-and-formatting

Mar 8, 2023

Contributor

MangoIV commented Mar 6, 2023

As talked about in #143, here are some linting and formatting checks for the nix code :)

Note
if the user chooses to use the devShell for development of this flake, this will
install a pre-commit-hook that will be required to be run before commiting, this can
be skipped with --no-verify (as stated in the README)

Warning
We now implicitly depend on flake-utils again, but perhaps this is not an issue...
Also note that we perhaps at some point want to check spelling in the REAMDE but I
didn't have time to setup hunspell for it


          [feat] add formatting and linting

6f8bccb

- formatting checks
- linting checks
- pre-commit-hooks
- corresponding devShell
- CI entry

supersven approved these changes

View reviewed changes

Collaborator

supersven left a comment

@MangoIV , Thanks a lot! This looks very useful. 👍

supersven merged commit d1375f4 into alpmestan:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet