Project

General

Profile

action #118588

coordination #96596: [qe-core][CI] CI/CD and Coding style improvements

[qe-core][CI] Anotate output when there are code violations on github's PR.

Added by szarate 5 months ago. Updated 2 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
pdostal
Category:
Infrastructure
Target version:
-
Start date:
2022-10-13
Due date:
% Done:

0%

Estimated time:
Difficulty:
Tags:
qe-core-october-sprint

Description

Whenever a check fails in the CI, it's often difficult to find directly the file that is causing the failure, for instance here.

At the moment things that come to mind are:

  1. Tidy checkss.
  2. Yaml lints.

Acceptance `Criteria

  1. Passing checks done on files, aren't shown unless --verbose or another flag are passed during the calls to make
Screenshot 2022-11-21 at 10.37.53.png (123 KB) Screenshot 2022-11-21 at 10.37.53.png pdostal, 2022-11-21 09:38
14130

Related issues

Copied to openQA Tests - action #122923: [qe-core][CI] Hide passing check messages from the CI logsResolved2022-10-13

History

#1 Updated by pdostal 5 months ago

  • Status changed from Workable to In Progress
  • Assignee set to pdostal

#2 Updated by pdostal 5 months ago

Here is simple test of github annotations: https://github.com/pdostal/test/pull/1/files

#3 Updated by pdostal 5 months ago

Here is perl-tidyall pull request.

#4 Updated by pdostal 5 months ago

Here is yamllint pull request.

#5 Updated by szarate 4 months ago

  • Sprint changed from QE-Core: October Sprint (Sep 28 - Oct 26) to QE-Core: November Sprint (Nov 09 - Nov 22)

#6 Updated by pdostal 4 months ago

Github annotation optimal line parameter bug report.

#7 Updated by pdostal 4 months ago

The perl-code-tidyall pull request.

#8 Updated by pdostal 4 months ago

The os-autoinst-distri-opensuse patch.

#9 Updated by pdostal 4 months ago

14130

Done for yamllint and tidyall.

#10 Updated by pdostal 4 months ago

The perlcritic pull request.

#11 Updated by szarate 2 months ago

  • Subject changed from [qe-core][CI] Hide passing checks from the CI to [qe-core][CI] Anotate output when there are code violations on github's PR.

Adjusting the name to something more fitting

#12 Updated by szarate 2 months ago

  • Copied to action #122923: [qe-core][CI] Hide passing check messages from the CI logs added

Also available in: Atom PDF