Project

General

Profile

Actions
Copy link

action #118588

closed

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 over 1 year ago. Updated over 1 year ago.

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

0%

Estimated time:
Difficulty:
Sprint:
QE-Core: November Sprint (Nov 09 - Dec 07)
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

Files

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

Related issues 1 (0 open — 1 closed)

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

Actions
Actions
Copy link
 #1

Updated by pdostal over 1 year ago

  • Status changed from Workable to In Progress
  • Assignee set to pdostal
Actions
Copy link
 #2

Updated by pdostal over 1 year ago

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

Actions
Copy link
 #3

Updated by pdostal over 1 year ago

Here is perl-tidyall pull request.

Actions
Copy link
 #4

Updated by pdostal over 1 year ago

Here is yamllint pull request.

Actions
Copy link
 #5

Updated by szarate over 1 year ago

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

Updated by pdostal over 1 year ago

Github annotation optimal line parameter bug report.

Actions
Copy link
 #7

Updated by pdostal over 1 year ago

The perl-code-tidyall pull request.

Actions
Copy link
 #8

Updated by pdostal over 1 year ago

The os-autoinst-distri-opensuse patch.

Actions
Copy link
 #9

Updated by pdostal over 1 year ago

Done for yamllint and tidyall.

Actions
Copy link
 #10

Updated by pdostal over 1 year ago

The perlcritic pull request.

Actions
Copy link
 #11

Updated by szarate over 1 year 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

Actions
Copy link
 #12

Updated by szarate over 1 year ago

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

Also available in: Atom PDF