Repository navigation
chore: align CI and bump test dependencies - #23
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: abaczek <arkadiusz.baczek@intel.com>
There was a problem hiding this comment.
Pull request overview
Aligns CI workflow wrappers in this repo with the templates used in intel/mfd-connect (removing redundant secrets/with inputs that the reusable workflows now derive themselves, and converting codeql.yml from an inline CodeQL setup into a thin wrapper over intel/mfd/.github/workflows/codeql.yml@main) and bumps the test toolchain.
Changes:
- Bump test deps:
pytestto>= 9.0.3, < 10andmfd-code-qualityto>= 1.4.1, < 2. - Trim reusable-workflow wrappers (
run_tests.yml,pull_request.yml,main.yml,manual_release.yml,check_code_standard.yml,check_pr_format.yml) to match the upstream template. - Replace inline CodeQL workflow with the reusable
intel/mfd/.github/workflows/codeql.yml@mainwrapper.
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| requirements-test.txt | Bump pytest constraint to >= 9.0.3, < 10. |
| requirements-dev.txt | Bump mfd-code-quality lower bound to 1.4.1. |
| .github/workflows/run_tests.yml | Drop secrets, REPOSITORY_NAME, BRANCH_NAME inputs from wrapper. |
| .github/workflows/pull_request.yml | Reformat existing inputs/secrets block. |
| .github/workflows/main.yml | Reformat existing inputs/secrets block. |
| .github/workflows/manual_release.yml | Reformat existing inputs/secrets block. |
| .github/workflows/check_code_standard.yml | Drop GH_TOKEN secret; add PYTHON_VERSION input. |
| .github/workflows/check_pr_format.yml | Reformat existing inputs block. |
| .github/workflows/codeql.yml | Replace inline CodeQL setup with reusable workflow wrapper. |
| .github/dependency_review.yml | Whitespace-only change on trailing line. |
Note: Across every modified workflow, the GitHub Actions expressions are written as ${{ ... } (single closing brace) instead of ${{ ... }}. As shown, this is invalid expression syntax and will break the workflows. If this is an artifact of how the diffs were rendered for review rather than the actual file content, please ignore — but the file content I read from disk for run_tests.yml and codeql.yml does show the single brace.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: abaczek <arkadiusz.baczek@intel.com>
2056164 to
a8c8b26
Compare
|
We don't publish DEVs .whl. |
Summary
Validation
git diff --check.github/**/*.ymlpytest -q