action #155062
closedopenQA Tests (public) - coordination #96596: [qe-core][CI] CI/CD and Coding style improvements
Unify GitHub Actions for QA Projects - perltidy in os-autoinst size:M
0%
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¶
- Follow the existing pull requests
- Ensure that os-autoinst as an example works with the updated os-autoinst-common changes
- Note that the existing tools/tidy is used from os-autoinst-distri-opensuse via cloning os-autoinst. And then
tidy
uses readlink to find out the right cpanfile to get the perltidy version. So tools/tidy shouldn't be deleted before os-autoinst-distri-opensuse is also using os-autoinst-common
Updated by okurz 11 months ago
- Copied from action #138416: Unify GitHub Actions for QA Projects size:M added
Updated by okurz 10 months ago
- Copied to action #155188: Unify GitHub Actions for QA Projects - perltidy&perlcritic in openQA added
Updated by livdywan 10 months ago
https://github.com/os-autoinst/os-autoinst/pull/2461 seems to have broken package builds on OBS
Updated by tinita 10 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)
Updated by tinita 10 months ago
https://github.com/os-autoinst/os-autoinst/pull/2462 t: Split author tests and build test
Updated by tinita 10 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)