Repository navigation
Refactor all occurences of non-contant format string err - #16690
Conversation
|
/lgtm |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #16690 +/- ##
==========================================
+ Coverage 80.29% 80.32% +0.02%
==========================================
Files 217 217
Lines 13560 13560
==========================================
+ Hits 10888 10892 +4
+ Misses 2309 2304 -5
- Partials 363 364 +1 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
@gauron99 I'll try to move the needle on other repos due to |
|
/test istio-latest-no-mesh |
|
@evankanderson PTAL if you happen to go by, thanks a lot! The PR's fixing Afterwards k8s 1.36 update should be smoother without formatting error noise. |
|
/cc @knative/steering-committee the above comment ^^ |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
No unresolved review comments remain, and all reviewers assessed the changes as approval-ready.
Review effort: Lite
Findings: None
What changed in this PR
Refactors lifecycle condition messages to use constant format strings for Go 1.26 compatibility.
Changes:
- Uses
%splaceholders for dynamic messages. - Removes unnecessary
fmtusage. - Preserves condition-message propagation.
| File | Description |
|---|---|
pkg/apis/serving/v1beta1/domainmapping_lifecycle.go |
Updates DomainMapping condition formatting |
pkg/apis/serving/v1/service_lifecycle.go |
Updates Service condition formatting |
pkg/apis/serving/v1/route_lifecycle.go |
Updates Route condition formatting |
pkg/apis/serving/v1/revision_lifecycle.go |
Updates Revision condition formatting |
pkg/apis/serving/k8s_lifecycle.go |
Updates Deployment condition propagation |
pkg/apis/autoscaling/v1alpha1/pa_lifecycle.go |
Updates PodAutoscaler condition formatting |
pkg/apis/autoscaling/v1alpha1/metric_lifecycle.go |
Updates Metric condition formatting |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
/lgtm |
|
/assign @dprotaso |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aliok, dsimansk The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/override istio-latest-no-mesh-tls_serving_main |
|
@aliok: Overrode contexts on behalf of aliok: istio-latest-no-mesh-tls_serving_main DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Fixes #
Proposed Changes
Release Note
Prep for go 1.26 to avoid error like:
knative/pkg#3392
/cc @dprotaso @gauron99