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 to installation instructions #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonasbn
Copy link
Contributor

@jonasbn jonasbn commented Feb 18, 2022

Updated installation documentation based on the deprecation of go get for executables with Go version 1.17

REF: https://go.dev/doc/go-get-install-deprecation

Copy link
Contributor

@subtlepseudonym subtlepseudonym left a comment

Choose a reason for hiding this comment

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

Looks good to me!

I do have the same nitpick about the commit message formatting, but it's not a blocker. For example, something like the message below better fits convention:

Update install instructions to use "go install"

The use of "go get" for installing go executables is deprecated as of
version 1.17: https://go.dev/doc/go-get-install-deprecation

The use of "go get" for installing go executables is deprecated as of
version 1.17: https://go.dev/doc/go-get-install-deprecation
@eduardosanzb
Copy link

lgtm!

@kbudde
Copy link

kbudde commented Oct 25, 2023

Just had the same issue. i can confirm go install works :)

@GRASBOCK GRASBOCK mentioned this pull request Mar 19, 2024
@subtlepseudonym subtlepseudonym mentioned this pull request Apr 15, 2024
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.

4 participants