Skip to content

Upgrade to Golang Version 1.22.6 and Golangci lint version v1.61.0#3988

Merged
gongmax merged 25 commits intoagones-dev:mainfrom
0xaravindh:golangci-lint-version-update
Oct 3, 2024
Merged

Upgrade to Golang Version 1.22.6 and Golangci lint version v1.61.0#3988
gongmax merged 25 commits intoagones-dev:mainfrom
0xaravindh:golangci-lint-version-update

Conversation

@0xaravindh
Copy link
Copy Markdown

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug

/kind cleanup

/kind documentation
/kind feature
/kind hotfix
/kind release

What this PR does / Why we need it:

Which issue(s) this PR fixes: #3949

Closes #3949

Special notes for your reviewer:

@github-actions github-actions bot added kind/cleanup Refactoring code, fixing up documentation, etc size/XL labels Sep 17, 2024
@github-actions
Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 67ba3955-fb92-4676-bf05-3c83802b6a08

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: e12c0ed2-f2c2-41a3-a2e4-e6e60a3d4892

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@0xaravindh 0xaravindh marked this pull request as ready for review September 19, 2024 08:54
@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 6c603335-9d25-4523-adc5-4c66220d84e7

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 0a32a258-5f3e-4fbc-bf0a-5ecdfb33f86e

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

go.mod Outdated
module agones.dev/agones

go 1.21
go 1.22.0
Copy link
Copy Markdown
Collaborator

@peterzhongyi peterzhongyi Sep 23, 2024

Choose a reason for hiding this comment

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

Does this need to be 1.22 instead of 1.22.0?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

updated @peterzhongyi

@github-actions
Copy link
Copy Markdown

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 51bc6965-3c3d-4b41-a2d0-52cce6e257f3

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 1, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 7de13ef0-54d5-4538-b972-af6ab21a2b93

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 1, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 6956a2d6-510d-43da-a3c4-1086cf05f866

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 1, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 8da229b7-744f-443f-a8fa-51e22435926c

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 2, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: 290c6dd1-aafe-4bd5-b133-f13d742d236a

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 2, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Failed 😭

Build Id: b1a6b6d2-a65c-440b-b8e8-a52ac5d415b6

Status: FAILURE

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@0xaravindh 0xaravindh closed this Oct 3, 2024
@0xaravindh 0xaravindh reopened this Oct 3, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 3, 2024

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@agones-bot
Copy link
Copy Markdown
Collaborator

Build Succeeded 🥳

Build Id: 6263372e-cc1a-48f1-aa83-262263365f03

The following development artifacts have been built, and will exist for the next 30 days:

A preview of the website (the last 30 builds are retained):

To install this version:

git fetch https://github.com/googleforgames/agones.git pull/3988/head:pr_3988 && git checkout pr_3988
helm install agones ./install/helm/agones --namespace agones-system --set agones.image.registry=us-docker.pkg.dev/agones-images/ci --set agones.image.tag=1.44.0-dev-17896ed

@gongmax gongmax changed the title Golangci lint version upgrade v1.61.0 Upgrade to Golang Version 1.22.6 and Golangci lint version v1.61.0 Oct 3, 2024
Copy link
Copy Markdown
Collaborator

@gongmax gongmax left a comment

Choose a reason for hiding this comment

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

LGTM. Some minor comments for my learning purpose.

@gongmax gongmax merged commit cd3bc13 into agones-dev:main Oct 3, 2024
@0xaravindh 0xaravindh self-assigned this Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/cleanup Refactoring code, fixing up documentation, etc size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade to Golang 1.22.6

5 participants