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

Update go version to 1.21 as 1.18 is causing issue in AppendUint32 #91

Merged
merged 14 commits into from
Oct 15, 2023

Conversation

sagar2395
Copy link

Issue #90 - Updated go version to 1.21.3 as 1.18 is causing issue in AppendUint32.…
Also fixed lint issues by updating required packages

@auto-assign auto-assign bot requested a review from edoardottt October 14, 2023 09:37
Copy link
Owner

@edoardottt edoardottt left a comment

Choose a reason for hiding this comment

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

golangci-lint is failing, maybe go1.21 is the correct format

@sagar2395
Copy link
Author

sagar2395 commented Oct 14, 2023

@edoardottt Updated to 1.21 version. Let's try using this version. Otherwise, I found another issue(golang/go#61851) in which it is mentioned that 1.20 worked fine. If this fails, I can update to that.

@edoardottt edoardottt changed the base branch from main to devel October 14, 2023 11:04
@edoardottt edoardottt linked an issue Oct 14, 2023 that may be closed by this pull request
Copy link
Owner

@edoardottt edoardottt left a comment

Choose a reason for hiding this comment

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

multiple different Go versions

@edoardottt edoardottt changed the title Update go version to 1.21.3 as 1.18 is causing issue in AppendUint32.… Update go version to 1.21 as 1.18 is causing issue in AppendUint32 Oct 14, 2023
@sagar2395
Copy link
Author

@edoardottt version updated for golangci-lint.

@@ -18,7 +18,7 @@ jobs:
steps:
- uses: actions/setup-go@v3
with:
go-version: 1.17
go-version: 1.21
Copy link
Owner

Choose a reason for hiding this comment

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

golangci-lint is failing, I guess is related to this golangci/golangci-lint#3933 (comment)

Copy link
Author

Choose a reason for hiding this comment

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

Updated golangci-lint to 1.54.2. This seems to work as per golangci/golangci-lint#3107 (comment)

@sagar2395
Copy link
Author

Updated golangci-lint to 1.54.2. This seems to work as per golangci/golangci-lint#3107 (comment)

Copy link
Owner

@edoardottt edoardottt left a comment

Choose a reason for hiding this comment

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

lgtm

@edoardottt edoardottt merged commit 36f7197 into edoardottt:devel Oct 15, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate why actions are failing
2 participants