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

docs: add version pinning information to the README #189

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

jedevc
Copy link
Contributor

@jedevc jedevc commented Jan 25, 2023

This should provide an easy reference for users starting to use this action who want to pin versions, so that they can easily identify the different versioning pattern used by this action.

@jedevc jedevc requested a review from crazy-max as a code owner January 25, 2023 12:38
@jedevc jedevc force-pushed the version-pinning-docs branch from f569ea7 to 95eafea Compare January 25, 2023 12:53
crazy-max
crazy-max previously approved these changes Jan 25, 2023
@crazy-max crazy-max dismissed their stale review January 25, 2023 13:03

Update TOC

Copy link
Member

@crazy-max crazy-max left a comment

Choose a reason for hiding this comment

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

Almost forgot, can you update the TOC?

This should provide an easy reference for users starting to use this
action who want to pin versions, so that they can easily identify the
different versioning pattern used by this action.

Signed-off-by: Justin Chadwell <me@jedevc.com>
@jedevc jedevc force-pushed the version-pinning-docs branch from 95eafea to 70deadb Compare January 25, 2023 13:07
@jedevc
Copy link
Contributor Author

jedevc commented Jan 25, 2023

Oops, my bad, done 🎉

@crazy-max crazy-max merged commit 7932f62 into master Jan 25, 2023
@crazy-max crazy-max deleted the version-pinning-docs branch January 25, 2023 13:11
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.

None yet

2 participants