-
Notifications
You must be signed in to change notification settings - Fork 15
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
chore: update readme when a new release is available #548
Conversation
e13bf3e
to
f957c24
Compare
3123d83
to
2eec1f6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, looks like a good change 👍
Can we have a linter which ensures this file stays correct? Otherwise people (like me) will forget. 😢
Yeah, I want to create a new PR with the remaining actions and then create that lint rule if that's okay? Assuming you're talking about the config file 🙃 |
OK if you want to do it that way, that's fine |
Co-authored-by: Iain Lane <[email protected]>
Co-authored-by: Iain Lane <[email protected]>
Co-authored-by: Iain Lane <[email protected]>
Co-authored-by: Iain Lane <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hell yeah
We need this feature so the README.md guide always contains the latest version of the action.
Currently our workflow READMEs are a bit misleading since we tell people to use
main
, which will cause backwards compatibility issues.Working example.