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

readme: fix invalid action tag in main snippet #161

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

Tropicao
Copy link
Contributor

Current snippet in README is missing the "v" prefix when targeting the action version, resulting in an early failure when trying to open a pull request on a repository with the corresponding action freshly installed.

While at it, use the latest tag.

Current snippet in README is missing the "v" prefix when targeting the
action version, resulting in an early failure when trying to open a pull
request on a repository with the corresponding action freshly installed.

While at it, use the latest tag.
Copy link
Contributor

@kingthorin kingthorin left a comment

Choose a reason for hiding this comment

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

For whatever it's worth. This does seem to be a valid and needed fix.

Copy link
Member

@ibakshay ibakshay left a comment

Choose a reason for hiding this comment

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

LGTM!

@ibakshay ibakshay merged commit bd7b55d into contributor-assistant:master Sep 17, 2024
4 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.

3 participants