-
Notifications
You must be signed in to change notification settings - Fork 572
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
fix: use the correct controller variable. #615
Conversation
size-limit report 📦
|
Codecov Report
@@ Coverage Diff @@
## main #615 +/- ##
==========================================
+ Coverage 76.28% 76.49% +0.22%
==========================================
Files 43 43
Lines 1454 1450 -4
Branches 284 283 -1
==========================================
Hits 1109 1109
+ Misses 313 310 -3
+ Partials 32 31 -1
Continue to review full report at Codecov.
|
ParametersDetails
Result
Details
Screenshots |
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.
Good catch but I don't think this is even being called 🤔
Yeah, you are right, it's not, let's do some clean-up then. |
* Update golang version to 1.19.8 * Add script to update go versions * Remove build matrix We no longer use to build other versions or OSes. This avoids having to change the required CI checks for protected branches on every go release. * Remove accidental curly braces that prevent caching
No description provided.