Skip to content

ci: speed up Python checks by running ruff/mypy without waiting for setup#1078

Merged
raman325 merged 1 commit intomainfrom
ci/speedup-checks
Apr 22, 2026
Merged

ci: speed up Python checks by running ruff/mypy without waiting for setup#1078
raman325 merged 1 commit intomainfrom
ci/speedup-checks

Conversation

@raman325
Copy link
Copy Markdown
Owner

Proposed change

Removes the needs: setup dependency from the ruff and mypy jobs so they start immediately in parallel with setup, instead of waiting ~30s for the version matrix computation they don't need.

Also:

  • Merge CI-only pip installs (pytest-cov, pytest-github-actions-annotate-failures) into the main install step
  • Optimize apt-get with --no-install-recommends

Before: setup → (ruff | mypy | pytest) — ruff/mypy waited for setup
After: (ruff | mypy | setup) → pytest — ruff/mypy start immediately

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (which adds functionality)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

  • This PR fixes or closes issue:
  • This PR is related to issue:

Copilot AI review requested due to automatic review settings April 22, 2026 05:27
@github-actions github-actions Bot added github-config Changes to .github/ configuration files code-quality Pull requests that improve code quality labels Apr 22, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR optimizes the Python CI workflow by allowing static analysis jobs to start immediately (in parallel with the version-matrix setup job), reducing overall wall-clock time for checks.

Changes:

  • Removed the needs: setup dependency from the ruff and mypy jobs and switched them to use env.TARGET_PYTHON.
  • Consolidated CI-only Python test dependencies into the main dependency install step.
  • Streamlined system package installation with apt-get install -y --no-install-recommends.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@raman325 raman325 merged commit 28e8e14 into main Apr 22, 2026
17 checks passed
@raman325 raman325 deleted the ci/speedup-checks branch April 22, 2026 05:29
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.66%. Comparing base (9b9f49c) to head (3967917).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1078   +/-   ##
=======================================
  Coverage   95.66%   95.66%           
=======================================
  Files          46       46           
  Lines        5279     5279           
  Branches      503      503           
=======================================
  Hits         5050     5050           
  Misses        229      229           
Flag Coverage Δ
python 97.32% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-quality Pull requests that improve code quality github-config Changes to .github/ configuration files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants