-
Notifications
You must be signed in to change notification settings - Fork 4
CI: test new "slim" runner variants #141
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
Merging this PR will improve performance by 11.08%
Performance Changes
Comparing |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #141 +/- ##
=======================================
Coverage 88.29% 88.29%
=======================================
Files 3 3
Lines 94 94
=======================================
Hits 83 83
Misses 11 11 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this 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 migrates GitHub Actions CI workflows from standard ubuntu-latest runners to new ubuntu-slim runner variants for testing purposes. As noted in the description, packaging/publishing workflows and benchmark workflows are intentionally excluded for security and performance reasons respectively.
Changes:
- Updated 3 jobs in the Python CI workflow (lint, test, codecov) to use
ubuntu-slim - Updated the deploy job in the docs workflow to use
ubuntu-slim - Updated the dependency review workflow to use
ubuntu-slim
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .github/workflows/python-ci.yaml | Migrated all three jobs (lint, test, codecov) from ubuntu-latest to ubuntu-slim |
| .github/workflows/docs.yaml | Migrated the deploy job from ubuntu-latest to ubuntu-slim |
| .github/workflows/dependency-review.yaml | Migrated the dependency-review job from ubuntu-latest to ubuntu-slim |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Bradley Reynolds <bradley.reynolds@tailstory.dev>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Packaging and publishing was left out for security reasons, and benchmarks were skipped for now for performance reasons.