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

add fmt and clippy check to CI #34

Merged

trobro merged 1 commit into hjson:master from gruebel:add-lint

Aug 3, 2024

Contributor

gruebel commented Aug 3, 2024

I disabled 2 of the default clippy lints match_like_matches_macro and needless_doctest_main. I was not sure, if you like the matches!(...) macro style or not and the second relates to the main example, so kept it as is.
if you don't like specific changes, then let me know and I can revert them and either disable the lint or exclude it specifically there


          add fmt and clippy check to CI

3a6c535

Member

trobro commented Aug 3, 2024

Thanks for the PR, looks good.

trobro merged commit c507a23 into hjson:master

5 checks passed

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

Labels

None yet