1
0
Fork 0
Commit graph

21 commits

Author SHA1 Message Date
Mikhail Goncharov
144354bc04 Generate HTML summary instead of adding comment
+ reworked phab-talk, now it's a class instead of free functions
2020-02-06 16:49:42 +01:00
Mikhail Goncharov
0519b5a4e3 Attach artifacts to build
Generate test summary HTML
2020-02-06 14:24:31 +01:00
Christian Kühnel
d75abdd172 fixed type errors 2020-01-30 17:45:57 +01:00
Mikhail Goncharov
562adb1b90 fix format string 2020-01-23 18:01:07 +01:00
Mikhail Goncharov
429c70ddc9 fix links 2020-01-23 14:13:42 +01:00
Mikhail Goncharov
a3f6610df2 Update bot report
- add links to join beta and report issue
- add link "not useful" to clang-tidy warning
- clang-tidy comment in report now tells how many inline comments were
  added
2020-01-23 13:35: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
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
8d390ff370 Refactor Phabricator report construction
Now report is constructed by adding more data from several stages and
represents internal structure rather what API expects.

+ Made all function that don't interact with phabricator free standing
+ Now links list all files in results directory
+ Status icons

Example comment: https://reviews.llvm.org/D71197#1779176 (links don't work).
2019-12-11 14:36:20 +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
Christian Kühnel
0be0498203 fix handling of compile errors
addresses #63
2019-11-21 15:04:39 +01:00
Christian Kühnel
db831d4c0e only posting non-empty comments 2019-11-18 16:22:43 +01:00
Christian Kühnel
c96a2d158c making feedback upload more relaxed if files are missing 2019-11-18 16:00:14 +01:00
Christian Kühnel
48567c02fe added retry on errors 2019-11-05 09:31:43 +01:00
Christian Kühnel
ba0f07ffb4 added status output 2019-10-24 09:41:19 -07:00
Christian Kühnel
a6abec98d4 handling missing test result file as failted build 2019-10-24 04:41:57 -07:00
Christian Kühnel
ca68a7f47d improved build result report 2019-10-17 17:11:17 +02:00
Christian Kühnel
e46b8172e0 added help for weird API 2019-10-16 13:52:29 +02:00
Christian Kühnel
b67aeb4714 added host parameter 2019-10-16 11:54:38 +02:00
Christian Kühnel
539dacd5b5 posting text comments and test results to Phabricator works #14 2019-10-15 18:12:19 +02:00
Christian Kühnel
4cb862a908 backup for github launch 2019-10-14 09:07:34 +02:00