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

Fix GoDoc link at the top of the README file #2387

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

ananthb
Copy link
Contributor

@ananthb ananthb commented Apr 26, 2024

I was really puzzled by the broken documentation at the current link. This version bump should fix it.

Copy link
Collaborator

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

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

Thanks!

Could you update the links further, to point directly at the pkg.go.dev host name?

@ananthb
Copy link
Contributor Author

ananthb commented Apr 26, 2024

I created the badge link again with their badge generation tool.

@ananthb ananthb requested a review from mtrmac April 26, 2024 20:44
@ananthb
Copy link
Contributor Author

ananthb commented Apr 26, 2024

I also cleaned up the README language and links a bit here: 9bc780a.

I can add that commit to this PR if that works for you.

Copy link
Collaborator

@mtrmac mtrmac left a comment

Choose a reason for hiding this comment

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

Thanks!

@mtrmac
Copy link
Collaborator

mtrmac commented Apr 29, 2024

I also cleaned up the README language and links a bit here

Some parts of that (like updating for the link redirect) are clearly valuable in isolation.

OTOH parts of the README description are to a large extent a historical artifact of how this library was split from skopeo, and should ideally be replaced wholesale — so incremental improvements to the wording would not be the best use of anyone’s time. (Not that I can immediately prioritize those wholesale replacements, either…)

@mtrmac mtrmac merged commit d7adcab into containers:main Apr 29, 2024
10 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