Zulip Chat Archive

Stream: mathlib4

Topic: CI "check for noisy stdout lines"


Jon Eugster (Oct 23 2023 at 11:01):

I found the CI message here not very helpful:
CI-error.png

It took me a moment to figure out that this was because of a linter warning:

linter-warning.png

Could we improve this? I wonder if simply adding -A1 to the grep search would already do something?

Alex J. Best (Oct 23 2023 at 12:26):

Adding A1 would probably help quite a bit yes! If you make a PR I will review it. Its a bit tricky to do better because I don't think we get line numbers or anything for stdout lines so I dont think the CI can pin the message anywhere more sensible, even finding which file it comes from is annoying when the build is parallel

Jon Eugster (Oct 23 2023 at 14:57):

#7868


Last updated: Dec 20 2023 at 11:08 UTC