Update version for codegen container#6717
Conversation
Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #6717 +/- ##
==========================================
+ Coverage 27.51% 29.54% +2.02%
==========================================
Files 554 592 +38
Lines 59393 63413 +4020
==========================================
+ Hits 16343 18734 +2391
- Misses 41792 43231 +1439
- Partials 1258 1448 +190
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I found an issue in this rabbit hole :(
Should we temporarily mark the lint/go workflow as not required in order to update go version for modules first, then re-enable it later? cc @Warashi @mohammedfirdouss |
Yes this is fine @khanhtc1202 as long as we have a single tracking PR that lists all affected modules and re-enabling lint is the last step before closing it. |
|
@Warashi @mohammedfirdouss Temporary removed go-lint-complete as required workflow in order to merge this PR. I will re-enable it later |
|
Changed my mind, should make code/gen step as not required instead of this lint+test steps. WIll check the behavior via PR #6711 |
Pull request was closed
What this PR does:
part of updating go version for pipecd to v1.26.2
Why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: