Project

General

Profile

Actions

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 about 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
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)

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


Related issues 1 (0 open1 closed)

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

Actions
Actions #1

Updated by pdostal about 2 years ago

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

Updated by pdostal about 2 years ago

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

Actions #3

Updated by pdostal about 2 years ago

Here is perl-tidyall pull request.

Actions #4

Updated by pdostal about 2 years ago

Here is yamllint pull request.

Actions #5

Updated by szarate about 2 years ago

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

Updated by pdostal about 2 years ago

Github annotation optimal line parameter bug report.

Actions #7

Updated by pdostal about 2 years ago

The perl-code-tidyall pull request.

Actions #8

Updated by pdostal about 2 years ago

The os-autoinst-distri-opensuse patch.

Actions #9

Updated by pdostal about 2 years ago

Done for yamllint and tidyall.

Actions #10

Updated by pdostal almost 2 years ago

The perlcritic pull request.

Actions #11

Updated by szarate almost 2 years 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 #12

Updated by szarate almost 2 years ago

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

Also available in: Atom PDF