action #122923
coordination #96596: [qe-core][CI] CI/CD and Coding style improvements
[qe-core][CI] Hide passing check messages from the CI logs
0%
Description
Whenever a check fails in the CI, it's often difficult to directly find the file causing the failure, for instance here.
At the moment, things that come to mind are:
- Tidy checks.
- Metadata checks
[yaml checks]
Acceptance Criteria¶
Passing checks done on files aren't shown unless
--verbose
or another flag is passed during the calls tomake
See also: https://github.com/os-autoinst/os-autoinst/pull/2235 for silencing Tidy
Related issues
History
#1
Updated by szarate 2 months ago
- Copied from action #118588: [qe-core][CI] Anotate output when there are code violations on github's PR. added
#7
Updated by rfan1 about 1 month ago
- Status changed from In Progress to Feedback
PR is merged, Let me monitor the following commits to see if any issues.
I don't change the logic for yamllint
CI checks for now, we can file a new ticket and check with the maintainer to see if we can do the same thing for it.
#8
Updated by rfan1 about 1 month ago
- Status changed from Feedback to Resolved
#9
Updated by pdostal about 1 month ago
rfan1 wrote:
PR is merged, Let me monitor the following commits to see if any issues.
Can you link the PR?
I don't change the logic for
yamllint
CI checks for now, we can file a new ticket and check with the maintainer to see if we can do the same thing for it.
Go for it, just let me know in the pull request.
#10
Updated by rfan1 about 1 month ago
pdostal wrote:
rfan1 wrote:
PR is merged, Let me monitor the following commits to see if any issues.
Can you link the PR?
oops, I missed the PR https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/16213I don't change the logic for
yamllint
CI checks for now, we can file a new ticket and check with the maintainer to see if we can do the same thing for it.Go for it, just let me know in the pull request.
Sure, I will ask Santiago's help to file a new ticket
#11
Updated by szarate about 1 month ago
- Sprint changed from QE-Core: January Sprint (Jan 11 - Feb 08) to QE-Core: February Sprint (Feb 08 - Mar 08)
- Tags set to qe-core-february-sprint
- Status changed from Resolved to Feedback
Reopened during the Planning, Richard has some more ideas on where things can improve
#12
Updated by rfan1 about 1 month ago
- Status changed from Feedback to In Progress
#13
Updated by rfan1 about 1 month ago
New PR for yaml check
https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/16356