-
-
Notifications
You must be signed in to change notification settings - Fork 4.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
devenv: upgrade colima to 0.7.5 #78881
Conversation
…or newer limactl version
6bb650d
to
868d5a0
Compare
devenv/config.ini
Outdated
@@ -1,5 +1,5 @@ | |||
[devenv] | |||
minimum_version = 1.10.2 | |||
minimum_version = 1.11.0 |
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.
this is a noop until minimum_version is implemented in 1.12.0 (getsentry/devenv#163)
but... 1.11.0 is a hard minimum requirement otherwise the newer colima won't be installed as a shim, and the newer limactl from homebrew can't be installed either
think we do need something in sync.py in the interim to lay the chicken egg
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.
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #78881 +/- ##
===========================================
+ Coverage 52.16% 78.42% +26.25%
===========================================
Files 7158 7139 -19
Lines 315942 315246 -696
Branches 43561 43338 -223
===========================================
+ Hits 164814 247218 +82404
+ Misses 149482 61707 -87775
- Partials 1646 6321 +4675 |
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.
actually, let's wait on merging this for a couple days so people can naturally update via |
sure thing |
see getsentry/devenv#146