Skip to content

ci: relocate Codecov config, pin actions, add concurrency#12

Open
thc1006 wants to merge 1 commit into
developfrom
chore/ci-harden
Open

ci: relocate Codecov config, pin actions, add concurrency#12
thc1006 wants to merge 1 commit into
developfrom
chore/ci-harden

Conversation

@thc1006

@thc1006 thc1006 commented Jun 27, 2026

Copy link
Copy Markdown
Member

Hardens and fixes the GitHub Actions setup (CI-config only, no source changes).

  • Relocate the misplaced Codecov config. .github/workflows/codecov.yml is a Codecov configuration file (it has no on:/jobs:), so Actions treats it as an invalid workflow and records a failed run on every commit. Moved to .github/codecov.yml (content unchanged). The real reusable workflow upload-to-codecov.yml correctly stays under workflows/.
  • Pin third-party actions off the moving @main/@master refs to current stable tags: checkout@v7, setup-python@v6, upload-artifact@v7, download-artifact@v8, codecov-action@v7, auto-assign@v3, gh-action-pypi-publish@v1.14.0 (supply-chain / stability hygiene).
  • Add concurrency cancel-in-progress to the PR-triggered Tests and Linters workflows so superseded runs are cancelled (deliberately not added to the scheduled slow workflow).
  • Pin ruff (ruff==0.15.20) in the lint workflow for reproducible results.

Note: master carries the same misplaced codecov.yml; a release or cherry-pick clears the permanent failure there too.

- Move .github/workflows/codecov.yml -> .github/codecov.yml. It is a Codecov
  config file (no on:/jobs:), so Actions parsed it as an invalid workflow and
  marked a failed run on every commit. The reusable upload-to-codecov.yml stays
  under workflows/.
- Pin third-party actions off moving @main/@master refs to current stable tags
  (checkout@v7, setup-python@v6, upload-artifact@v7, download-artifact@v8,
  codecov-action@v7, auto-assign@v3, gh-action-pypi-publish@v1.14.0).
- Add concurrency cancel-in-progress to the PR-triggered Tests and Linters
  workflows (not the scheduled slow one).
- Pin ruff (ruff==0.15.20) in the lint workflow for reproducible results.

CI-config only; no source changes.
@zuorenchen

Copy link
Copy Markdown
Member

Do you want to send this PR to RocketPy directly? We can then pull from RocketPy

@thc1006

thc1006 commented Jun 27, 2026

Copy link
Copy Markdown
Member Author

Do you want to send this PR to RocketPy directly? We can then pull from RocketPy

For sure and I will also submit the fixes for the other accelerated optimization issues. I am currently running benchmarks and tests.

@thc1006 thc1006 self-assigned this Jun 27, 2026
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