Repository navigation
Commit 1048e1e
committed
Annotate javac warnings from the build job only
setup-java's problem matcher turns javac's warnings, Error Prone's
among them, into annotations on the diff, and it is on by default. It
was on in every job that compiles the same sources: build, the three
newer JDKs, the five native platforms and both CodeQL analyses. Windows
never annotated, because the matcher's pattern takes no path with a
colon in it.
GitHub keeps only the first ten warnings of a step. The build prints
188, and as it runs in parallel, each job prints them in its own order
and keeps its own ten. So on #98 one warning showed up to ten times,
the one next to it once, and most not at all.
build keeps the matcher, now explicitly and with a comment saying why;
every other setup-java step on a pull request passes
problem-matcher: false. Each warning shows at most once, though still
only the first ten the build prints.1 parent b52adaf commit 1048e1e
2 files changed
Lines changed: 13 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
25 | 29 | | |
26 | 30 | | |
27 | 31 | | |
28 | 32 | | |
29 | 33 | | |
| 34 | + | |
30 | 35 | | |
31 | 36 | | |
32 | 37 | | |
| |||
68 | 73 | | |
69 | 74 | | |
70 | 75 | | |
| 76 | + | |
71 | 77 | | |
72 | 78 | | |
73 | 79 | | |
74 | 80 | | |
75 | 81 | | |
76 | 82 | | |
| 83 | + | |
77 | 84 | | |
78 | 85 | | |
79 | 86 | | |
80 | 87 | | |
81 | 88 | | |
82 | 89 | | |
83 | 90 | | |
| 91 | + | |
84 | 92 | | |
85 | 93 | | |
86 | 94 | | |
| |||
138 | 146 | | |
139 | 147 | | |
140 | 148 | | |
141 | | - | |
| 149 | + | |
| 150 | + | |
142 | 151 | | |
143 | 152 | | |
144 | 153 | | |
145 | 154 | | |
146 | 155 | | |
147 | 156 | | |
| 157 | + | |
148 | 158 | | |
149 | 159 | | |
150 | 160 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
| 33 | + | |
33 | 34 | | |
34 | 35 | | |
35 | 36 | | |
36 | 37 | | |
37 | 38 | | |
| 39 | + | |
38 | 40 | | |
39 | 41 | | |
40 | 42 | | |
| |||
0 commit comments