Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update docs to use go install instead of go get #227

Merged
merged 1 commit into from
Nov 8, 2023

Conversation

cbliard
Copy link
Contributor

@cbliard cbliard commented Nov 3, 2023

go get -u github.com/danielgtaylor/shorthand/cmd/j did not work for me. I ran it from a golang docker container because I do not have any go development environment available. Following error messages from the go command, I ended up with go install github.com/danielgtaylor/shorthand/cmd/j@latest which worked and installed the j executable in /go/bin/j.

Copy link
Owner

@danielgtaylor danielgtaylor left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link

codecov bot commented Nov 5, 2023

Codecov Report

Merging #227 (0a1f26c) into main (c4befd0) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #227   +/-   ##
=======================================
  Coverage   76.92%   76.92%           
=======================================
  Files          26       26           
  Lines        3666     3666           
=======================================
  Hits         2820     2820           
  Misses        639      639           
  Partials      207      207           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c4befd0...0a1f26c. Read the comment docs.

@danielgtaylor danielgtaylor merged commit b75a307 into danielgtaylor:main Nov 8, 2023
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants