-
Notifications
You must be signed in to change notification settings - Fork 2k
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
update-gopsutil, closes #10761 #10790
Conversation
cd08bbb
to
58e4de8
Compare
58e4de8
to
788a45d
Compare
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 fix LGTM. Any chance we can encourage gopsutil folks to cut a release tag for this? We do have a history of pointing to specific commits but we try to avoid adding to that because it upsets downstream packagers.
@tgross I don't think we should wait, especially if we can get this into v1.1.2 |
Good call, let's merge it. |
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
update to include shirou/gopsutil#1088, fixes slow cpu info for windows platform. closes #10761