1
0
Fork 0
Commit graph

11 commits

Author SHA1 Message Date
Christian Kühnel
4465b0fc52 changed output paths of lint.sh 2020-03-26 10:19:20 +01:00
Christian Kühnel
580d4ba63c fixed linter script
now support staging area and HEAD~1
2020-03-26 09:47:45 +01:00
Mikhail Goncharov
2fc3174a5c fix empty line removal for clang-tidy #136 2020-03-10 13:21:12 +01:00
Mikhail Goncharov
c1396546e5 Remove empty lines at the end of clang-tidy report 2020-02-19 16:17:50 +01:00
Mikhail Goncharov
563a115006 fix git diff #108 2020-01-24 09:06:40 +01:00
Mikhail Goncharov
276978ff1f Ignore selected paths for clang-format and clang-tidy
Now clang-format report will skip files matching `clang-format.ignore`.
For now it's empty.

clang-tidy will not receive diffs for `clang-tidy.ignore` (it still can
produce warnings for these files).
In addition build bot will not will not post comments for files
that are matched by `clang-tidy-comments.ignore`. Now project/file should
be whitelisted to receive inline comments from clang-tidy.

Added all /test directories to `clang-tidy.ignore`.

Other alternatives considered:

- using 'compile_commands.json': does not contain header files.
- specifying -regex option to 'clang-tidy-diff': probably not the best
  experience of providing multiple rules and maintaining a single regex.
2020-01-22 19:05:36 +01:00
Mikhail Goncharov
82c469fc69 Use default style for clang-format
It correctly picks up .clang-format settings.

For issue #92 and related.
2020-01-21 15:44:35 +01:00
Mikhail Goncharov
cac68d0d60 temporary disable clang-tidy 2020-01-13 11:05:24 +01:00
Mikhail Goncharov
453aa2ae84 Run clang-tidy and parse message
+ don't assume that clang-format has passed if there is no file.
  If linters didn't run at all file will be missing.
+ add "gray" icon for the result if it's unknown.
+ assume whole build is failed if clang-format or clang-tidy found something.
+ combine multiple lint messages per line.
2019-12-17 11:00:30 +01:00
Mikhail Goncharov
8e5db1e6bd Update image to use default clang package
+ add jq to parse conduit responses later
2019-12-11 15:58:00 +01:00
Mikhail Goncharov
7faaec98e7 Attach diffs produced by git-clang-format as lint messages.
E.g.: https://reviews.llvm.org/D71197?id=233029
Will add a link to file and code to apply the patch in the next PR.

+ don't create TARGET_DIR in scripts;
+ updated section about local build;
+ partially specified inputs / outputs of scripts so it's more transparent what are the results;
+ added symlink to compile_command.json (clang-tidy will need it);
+ add IDEA files to .gitignore.
2019-12-11 08:55:38 +01:00