update dependencies to use gopsutil v4#629
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the project’s process-inspection dependency to gopsutil v4 to eliminate the go-m1cpu transitive dependency implicated in macOS (M5) crashes (issue #628), and refreshes related module versions.
Changes:
- Migrate gopsutil imports from
github.com/shirou/gopsutil/v3/...to.../v4/...in the lock implementation and tests. - Update
go.mod/go.sumto requiregithub.com/shirou/gopsutil/v4and dropgithub.com/shoenig/go-m1cpu(plus related dependency updates).
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| lock/lock.go | Switches process PID existence checks to gopsutil v4 import path. |
| lock/lock_test.go | Updates tests to use gopsutil v4 process APIs. |
| go.mod | Bumps gopsutil to v4, removes go-m1cpu indirect dep, and updates related module versions. |
| go.sum | Reflects the updated dependency graph (adds v4-related sums; removes v3/go-m1cpu sums). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #629 +/- ##
==========================================
- Coverage 80.41% 79.55% -0.86%
==========================================
Files 150 170 +20
Lines 11796 12300 +504
==========================================
+ Hits 9485 9785 +300
- Misses 1827 2020 +193
- Partials 484 495 +11
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
update dependencies to use gopsutil from v3 to v4
gopsutil v4 version no longer needs
go-m1cpupackageFixes #628