Skip to content

ci: avoid duplicate PR checks and run full CDA tests nightly - #315

Merged
krowvin merged 3 commits into
mainfrom
ci/streamline-checks
Sep 10, 2026
Merged

ci: avoid duplicate PR checks and run full CDA tests nightly#315
krowvin merged 3 commits into
mainfrom
ci/streamline-checks

Conversation

@krowvin

@krowvin krowvin commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Avoid duplicate tests on feature-branch pushes and move expensive CDA coverage to nightly/manual runs. Keep the existing standalone workflows, without a custom policy script, shared setup action, or aggregate merge gate.

  • Unit tests and strict mypy run on Python 3.9 and latest stable (3.x) for all PRs and pushes to main. Formatting and CodeQL run on PRs/main; superseded PR runs are cancelled.
  • The full 18-job CDA matrix runs nightly at 08:17 UTC and through manual dispatch. It no longer runs automatically on PRs or merges, and no longer repeats mypy in every matrix job. Cached environments still run dependency installation with binary-only dependencies.
  • Keep exact-tag release testing and version validation before publishing, using latest stable Python. Preserve main's setup-python v7 upgrade.
  • Document the simple trigger policy and manual integration command. Draft and documentation PRs receive the same lightweight checks as other PRs; there is no changed-file classification.

Validation: actionlint, configured YAML formatting, and whitespace checks pass. Parsed-workflow assertions verify standalone triggers, only Python 3.9/latest, and exactly 18 nightly CDA combinations. Python application code and dependencies are unchanged; the earlier local Python 3.9/3.14 mock tests and type checks passed. GitHub checks will validate the revised workflow head.

Tradeoff: CDA integration regressions may be found after merge. Maintainers can run gh workflow run CDA-testing.yml --ref <branch> before merging when needed. Repository admins should require the two unit-test checks, formatting, CodeQL, and existing external checks; nightly CDA jobs must not be required on PRs. This PR does not change repository rules or publish a package.

@krowvin
krowvin requested a review from msweier September 10, 2026 19:41
@krowvin

krowvin commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator Author

Dialing this back a bit from the python scripts, they're a bit much and would be hard to maintain.

if for example someone wants to add a new doc extensions for doc deployment they might never think to go into a python script in the scripts directory to have this included.

@krowvin krowvin changed the title ci: streamline PR checks and schedule full CDA coverage ci: avoid duplicate PR checks and run full CDA tests nightly Sep 10, 2026

@msweier msweier left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! Maybe add a note to CONTRIBUTING.MD about gh workflow run CDA-testing.yml --ref <branch> so we remember to run it before approving something? Or if we want to check?

@sonarqubecloud

Copy link
Copy Markdown

@krowvin
krowvin merged commit e7a2dc8 into main Sep 10, 2026
8 checks passed
@krowvin
krowvin deleted the ci/streamline-checks branch September 10, 2026 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants