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

Add cpp linting and file checking to pre-commit config #8

Open
5 tasks
thclark opened this issue Jul 24, 2020 · 0 comments
Open
5 tasks

Add cpp linting and file checking to pre-commit config #8

thclark opened this issue Jul 24, 2020 · 0 comments
Assignees
Labels
devops To do with devops (test, CI/CD, gitops etc) good first issue Good for newcomers

Comments

@thclark
Copy link
Owner

thclark commented Jul 24, 2020

Add one or more of:

  • cpplint
  • clang-check
  • cppcheck

From https://gitlab.com/daverona/pre-commit-cpp to the pre-commit hooks that get invoked.

Since some windows users are getting involved (#6) make sure line endings etc are always consistent (although it's less of an issue nowadays with IDEs being sensible) using:

  • remove-crlf - Replace CRLF end-lines by LF ones before committing
  • remove-tabs - Replace tabs by whitespaces before committing

from https://github.com/Lucas-C/pre-commit-hooks

@thclark thclark self-assigned this Jul 24, 2020
@thclark thclark added devops To do with devops (test, CI/CD, gitops etc) feature Feature roadmapped for implementation good first issue Good for newcomers and removed feature Feature roadmapped for implementation labels Jul 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops To do with devops (test, CI/CD, gitops etc) good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant