Revamp CI workflow structure and job matrix #32
Merged
+61
−9
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
concurrencywith cancellation and narrow workflowpermissionstocontents: readfor safer run behavior.Description
testsjob with three jobs:lint(runsruff,black,isort),type-check(runsmypy), and a matrixtestjob (runspytest), updating steps to use a commoncheckout/setup/installpattern in.github/workflows/ci.yml.concurrencytocancel-in-progressruns on the same ref and restrictedpermissionstocontents: readin the workflow file.os: [ubuntu-latest, macos-latest, windows-latest]andpython-version: ["3.11", "3.12"]withfail-fast: falseand standardized dependency installation viapip install -r requirements-dev.txt.pytest --cov=veracrypt --cov-report=term-missing --cov-fail-under=80.Testing
pytest -q, which succeeded with42 passedtests.Codex Task