Skip to content

Optimize GitHub Actions environment setup#1156

Merged
ruck314 merged 12 commits into
pre-releasefrom
github-actions-opt
Apr 2, 2026
Merged

Optimize GitHub Actions environment setup#1156
ruck314 merged 12 commits into
pre-releasefrom
github-actions-opt

Conversation

@bengineerd

@bengineerd bengineerd commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

Description

Speed up GitHub Actions by reducing unnecessary dependency installation on normal pushes, separating documentation-only Python requirements, caching the macOS conda environment, and updating checkout/setup-python to Node 24-compatible action releases.

Details

  • Split the Python dependency lists into pip_requirements_ci.txt and pip_requirements_docs.txt so Linux push CI does not install Sphinx and other docs-only packages on every run.
  • Updated .github/workflows/rogue_ci.yml to fail fast on whitespace issues, defer documentation package installation to tag-only steps, and cancel superseded workflow runs.
  • Reworked the macOS job to cache conda packages and the rogue_build environment, and removed the redundant pip overlay install.
  • Updated actions/checkout and actions/setup-python to Node 24-compatible major versions to address the GitHub runner deprecation warning.
  • Validation performed locally: ruby -e 'require "yaml"; YAML.load_file(".github/workflows/rogue_ci.yml"); puts "YAML ok"' and git diff --check -- .github/workflows/rogue_ci.yml pip_requirements_ci.txt pip_requirements_docs.txt.

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.98%. Comparing base (455967d) to head (6321bb4).
⚠️ Report is 1079 commits behind head on pre-release.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@               Coverage Diff               @@
##           pre-release    #1156      +/-   ##
===============================================
+ Coverage        29.54%   29.98%   +0.43%     
===============================================
  Files               65       68       +3     
  Lines             6644     6977     +333     
  Branches          1015      973      -42     
===============================================
+ Hits              1963     2092     +129     
- Misses            4527     4736     +209     
+ Partials           154      149       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

bengineerd and others added 9 commits March 27, 2026 10:39
pytest-xdist is required for the -n auto --dist loadfile flags used in the CI test step.
softioc (pythonSoftIOC) is required by the epicsV7 protocol tests; without it builder is undefined and test_local_root fails.
Required for -n auto --dist loadfile flags used in the macOS CI test step.
@ruck314 ruck314 merged commit 489c79f into pre-release Apr 2, 2026
7 checks passed
@ruck314 ruck314 deleted the github-actions-opt branch April 2, 2026 15:27
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.

3 participants