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
Check code style during Travis CI build #100
Conversation
@floriankramer maybe we could combine this with doing just a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
I think this will at first annoy and then really help us:)
misc/format-check.sh
Outdated
printf "Use clang-format with the .clang-format provided in the QLever\n" | ||
printf "repository's root to ensure all code files are formatted " | ||
printf "properly.\n" | ||
printf "\x1b[m" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this was an issue as far as I remember it would be nice to tell which clang-format version must be used. (at least "must match the ones from the ubuntu18.04 repo" although explicit x.y.z would of course be nicer.
The first commit just adds the check without formatting so it breaks the build as intended