Skip to content
New issue

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

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

Already on GitHub? # to your account

Automatically run appropriate linters #18

Open
4 tasks
baileyparker opened this issue Mar 13, 2018 · 0 comments
Open
4 tasks

Automatically run appropriate linters #18

baileyparker opened this issue Mar 13, 2018 · 0 comments

Comments

@baileyparker
Copy link
Owner

baileyparker commented Mar 13, 2018

  • Valgrind (installed) for C/C++ (requires mucking with simple_test.runner)
    • Probably want to run as part of regular tests (run each test twice, second time for valgrind)
    • Probably want to send valgrind output to file (to not muck with stderr of compiler): valgrind: --log-filename="path/to/tmp/file"
  • go vet (installed), go fmt (installed), golint (not installed)
  • checkstyle (installed) for java with attached config: checkstyle.xml.gz (gzipped because GitHub doesn't support uploading XMLs, really GitHub?)
  • flake8 (installed) for Python

Probably need a mechanism for automatically detecting language (exclude integration_test dir!). Lists taken from: https://github.com/github/linguist/blob/master/lib/linguist/languages.yml

  • Python: *.py
  • C/C++: *.c, *.cpp, *.c++, *.cc, *.cxx, *.h, *.h++, *.hh, *.hpp, *.hxx
  • Java: *.java
  • Go: *.go

Also need flags to force language detection and disable this feature (part of making bin/run_harness a python script for #17).

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

No branches or pull requests

1 participant