-
Notifications
You must be signed in to change notification settings - Fork 20
add cpu info #691
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
add cpu info #691
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #691 +/- ##
=======================================
Coverage 96.29% 96.29%
=======================================
Files 111 111
Lines 6288 6288
=======================================
Hits 6055 6055
Misses 233 233 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 adds a new step in multiple GitHub workflow files to print CPU information during CI runs.
- Adds a “Print CPU information” step to task-unit-test, event-pull_request, and coverage workflows.
- The step attempts to use lscpu, or falls back to other commands based on the OS.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
File | Description |
---|---|
.github/workflows/task-unit-test.yml | Adds CPU info step with details for Linux and macOS. |
.github/workflows/event-pull_request.yml | Adds CPU info step for Linux using lscpu/fallback. |
.github/workflows/coverage.yml | Adds CPU info step for Linux using lscpu/fallback. |
@@ -34,6 +34,26 @@ jobs: | |||
uses: actions/checkout@v4 | |||
with: | |||
submodules: recursive | |||
- name: Print CPU information |
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.
The 'Print CPU information' step is duplicated across three workflows. Consider refactoring it into a reusable composite action to reduce duplication and simplify future maintenance.
Copilot uses AI. Check for mistakes.
Describe the changes in the pull request
Print the CPU info in our GHA flows
Mark if applicable