search

knative / test-infra

Test infrastructure for the Knative project
Apache License 2.0
82 stars 164 forks source link

[main] Format Go code #3818

Closed knative-automation closed 1 year ago

knative-automation commented 1 year ago

Cron -knative-prow-robot

/cc knative/productivity-wg-leads /assign knative/productivity-wg-leads

Produced by: knative-sandbox/knobots/actions/update-gofmt

Details:

knative-prow[bot] commented 1 year ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: knative-automation Once this PR has been reviewed and has the lgtm label, please assign dsimansk for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/knative/test-infra/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
dprotaso commented 1 year ago

/hold

@kvmware seems like the go linter and the format go knobots job differ

Linter wanted me to make the change earlier - https://github.com/knative/test-infra/pull/3815/files

I wonder if there is a drift in go versions or something

krsna-m commented 1 year ago

https://github.com/knative-sandbox/knobots/pull/302