-
Notifications
You must be signed in to change notification settings - Fork 2
rework CI jobs, cleanups #37
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
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
Reworks the CI setup and cleans up project scripts to streamline testing, caching, and dependency updates.
- Enhances
justfilewith new CI/test helpers, environment flags, and installation recipes - Restructures
ci.ymlto split jobs (test, nightly packaging, MSRV, coverage), add caching, and conditional release - Adds grouping in
dependabot.ymlfor GitHub Actions and Cargo dependency updates
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| justfile | Refactored CI/test targets, added env-info, get-crate-field, and improved installation recipes |
| .github/workflows/ci.yml | Split and renamed CI jobs, added caching steps, conditional flows, and a final pass check |
| .github/dependabot.yml | Introduced groups for version and security updates on Actions and Cargo |
Comments suppressed due to low confidence (2)
.github/workflows/ci.yml:35
- The job name "Test" is used twice (for default and nightly tests). Consider renaming this one to clarify its purpose (e.g.,
Test nightly package).
name: Test
.github/dependabot.yml:9
- The
groupskey appears misaligned under the update entry; align it withschedule(same indentation) so Dependabot recognizes the grouping configuration.
groups:
Codecov ReportAll modified and coverable lines are covered by tests ✅ 📢 Thoughts on this report? Let us know! |
|
Thank you for the rework and cleanup. |
No description provided.