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

set min go version to 1.21 #29

Merged
merged 2 commits into from
Dec 3, 2024
Merged

set min go version to 1.21 #29

merged 2 commits into from
Dec 3, 2024

Conversation

brainexe
Copy link
Contributor

@brainexe brainexe commented Dec 2, 2024

get rid of some deprecations

get rid of some deprecations
Copy link
Collaborator

@Felixoid Felixoid left a comment

Choose a reason for hiding this comment

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

I personally prefer https://github.com/pelletier/go-toml and moved all my projects there.

Although I done it a few years ago, when https://github.com/BurntSushi/toml looked very stale

@Felixoid
Copy link
Collaborator

Felixoid commented Dec 2, 2024

GO111MODULE=off go get golang.org/x/lint/golint
should be go install

And both upload-artifacts in

uses: actions/upload-artifact@v2
and below should be upgraded to v4. Could you do it?

fix some more outdated places
go.mod Show resolved Hide resolved
@Felixoid Felixoid mentioned this pull request Dec 3, 2024
@Felixoid Felixoid merged commit 6051ffc into leoleovich:master Dec 3, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants