Project

General

Profile

Actions
Copy link

action #155062

closed

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

Unify GitHub Actions for QA Projects - perltidy in os-autoinst size:M

Added by okurz 3 months ago. Updated about 2 months ago.

Status:
Resolved
Priority:
Low
Assignee:
tinita
Category:
Feature requests
Target version:
Ready
Start date:
2023-10-24
Due date:
% Done:

0%

Estimated time:
Tags:
platform-team

Description

Motivation

See #138416

Acceptance criteria

  • AC1: At least os-autoinst inherit perl tidy checks from os-autoinst-common
  • AC2: Checks in os-autoinst still run successfully on the master branch
  • AC3: os-autoinst-distri-opensuse tidy checks still work (as they are inherited from os-autoinst)

Suggestions

Related issues 2 (1 open — 1 closed)

Copied from openQA Project - action #138416: Unify GitHub Actions for QA Projects size:MResolvedybonatakis2023-10-24

Actions
Copied to openQA Project - action #155188: Unify GitHub Actions for QA Projects - perltidy&perlcritic in openQANew2023-10-24

Actions
Actions
Copy link
 #1

Updated by okurz 3 months ago

  • Copied from action #138416: Unify GitHub Actions for QA Projects size:M added
Actions
Copy link
 #2

Updated by tinita 3 months ago

  • Description updated (diff)
Actions
Copy link
 #3

Updated by tinita 3 months ago

  • Assignee set to tinita
Actions
Copy link
 #4

Updated by okurz 3 months ago

  • Priority changed from Normal to Low
Actions
Copy link
 #5

Updated by okurz 3 months ago

  • Copied to action #155188: Unify GitHub Actions for QA Projects - perltidy&perlcritic in openQA added
Actions
Copy link
 #6

Updated by okurz 3 months ago

  • Subject changed from Unify GitHub Actions for QA Projects - perltidy&perlcritic to Unify GitHub Actions for QA Projects - perltidy in os-autoinst size:M
  • Description updated (diff)
  • Status changed from New to Workable
Actions
Copy link
 #7

Updated by tinita 2 months ago

  • Status changed from Workable to In Progress
Actions
Copy link
 #9

Updated by tinita 2 months ago · Edited

https://github.com/os-autoinst/os-autoinst/pull/2461 git subrepo pull (merge) external/os-autoinst-common (merged)

which broke OBS builds because external/.../00-tidy.t was added and for some reason we run tests with no directory arguments by default. (and for some reason OBS checks were not run in this PR)

https://github.com/os-autoinst/os-autoinst/pull/2463 t: Explicitly only run tests under t/ and xt/ (merged)

Actions
Copy link
 #10

Updated by tinita 2 months ago

https://github.com/os-autoinst/os-autoinst/pull/2462 t: Split author tests and build test

Actions
Copy link
 #11

Updated by tinita about 2 months ago · Edited

  • Status changed from In Progress to Feedback

https://github.com/os-autoinst/os-autoinst/pull/2467 Add symlink to 00-tidy.t from os-autoinst-common (merged)

Actions
Copy link
 #12

Updated by tinita about 2 months ago

  • Status changed from Feedback to Resolved
Actions
Copy link

Also available in: Atom PDF