Skip to content

[Enhancement] Moved all workflows to run on a single env#34

Merged
psriramsnc merged 7 commits intomainfrom
scratch/ci_workflows
Sep 15, 2025
Merged

[Enhancement] Moved all workflows to run on a single env#34
psriramsnc merged 7 commits intomainfrom
scratch/ci_workflows

Conversation

@psriramsnc
Copy link
Collaborator

@psriramsnc psriramsnc commented Sep 15, 2025

📋 Summary

Consolidate all CI workflows.

✨ CI Workflow: Run CI

This PR introduces a consolidated CI workflow named Run CI that ensures consistent checks and documentation deployment across the repository.

🔧 Key Changes

  • Added a single setup job (setup_grasp) to warm the cache and prepare dependencies.
  • Added jobs for:
    • Format → runs make format to enforce code formatting.
    • Lint → runs make lint for static analysis.
    • Test → runs make test for unit and integration tests.
    • Docs Build → runs make docs to build project documentation.
    • Docs Deploy → automatically deploys documentation to GitHub Pages when changes are merged to main.

📦 Improvements

  • Reuses poetry virtual environment cache across jobs for faster builds.
  • Ensures consistent Python (3.9) setup across all jobs.
  • Protects GitHub Pages deployment by restricting to the main branch with environment rules.

✅ Outcome

  • Faster CI with warm caching.
  • Automatic formatting, linting, and testing on every PR.
  • Documentation is built and deployed seamlessly on main.

Performance impact (if any):

N/A

How to Test the feature

N/A

Screenshots (if applicable)

N/A

Checklist

  • Lint fixes and unit testing done
  • End to end task testing
  • Documentation updated

Notes

N/A

@psriramsnc psriramsnc marked this pull request as ready for review September 15, 2025 05:10
@psriramsnc psriramsnc requested a review from a team as a code owner September 15, 2025 05:10
@psriramsnc psriramsnc marked this pull request as draft September 15, 2025 05:17
@psriramsnc psriramsnc marked this pull request as ready for review September 15, 2025 05:28
@vipul-mittal vipul-mittal marked this pull request as draft September 15, 2025 07:10
@psriramsnc psriramsnc marked this pull request as ready for review September 15, 2025 09:03
@psriramsnc psriramsnc self-assigned this Sep 15, 2025
@psriramsnc psriramsnc added the enhancement New feature or request label Sep 15, 2025
@psriramsnc psriramsnc changed the title Moved all workflows to run on a single env [Enhancement] Moved all workflows to run on a single env Sep 15, 2025
@psriramsnc psriramsnc merged commit 181fa92 into main Sep 15, 2025
6 checks passed
@psriramsnc psriramsnc deleted the scratch/ci_workflows branch September 15, 2025 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants