Skip to content
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

Address golangci-lint v1.60.3 errors #1631

Merged
merged 2 commits into from
Aug 29, 2024

Conversation

tpantelis
Copy link
Contributor

Specifically, gosec "integer overflow conversion" issues.

Depends on submariner-io/shipyard#1715

@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr1631/tpantelis/v1.60.3_linting
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

Copy link
Member

@skitt skitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There’s no Shipyard bump in the affected module (it doesn’t depend on Shipyard).

@tpantelis
Copy link
Contributor Author

There’s no Shipyard bump in the affected module (it doesn’t depend on Shipyard).

yeah as it turned out. I had thought it would be affected by the Shipyard E2E changes but it wasn't. Still it doesn't hurt to bump it but I'll remove it.

@tpantelis tpantelis changed the title Bump shipyard and address golangci-lint v1.60.3 errors Address golangci-lint v1.60.3 errors Aug 28, 2024
Specifically, gosec "integer overflow conversion" issues.

Signed-off-by: Tom Pantelis <[email protected]>
Copy link

@tpantelis tpantelis requested a review from skitt August 29, 2024 12:32
@submariner-bot submariner-bot added the ready-to-test When a PR is ready for full E2E testing label Aug 29, 2024
@skitt skitt enabled auto-merge (rebase) August 29, 2024 12:39
@skitt skitt merged commit 724832a into submariner-io:devel Aug 29, 2024
23 checks passed
@submariner-bot
Copy link
Contributor

🤖 Closed branches: [z_pr1631/tpantelis/v1.60.3_linting]

@tpantelis tpantelis deleted the v1.60.3_linting branch August 30, 2024 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-test When a PR is ready for full E2E testing
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants