-
Notifications
You must be signed in to change notification settings - Fork 327
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
Add a compatibility table to the README #2273
Conversation
86af32d
to
37ba75b
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.
Thanks, this will help clear up something that has created confusion.
37ba75b
to
92c7c0d
Compare
Specifies the versions of the action, CLI and GHES that are jointly compatible.
92c7c0d
to
e50fb8f
Compare
Co-authored-by: Henry Mercer <henrymercer@github.com>
Oops. I just merged a previous PR that put the changelog note in the wrong place. Let me fix that. |
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.
It looks like there was a merge in the changelog file that didn't go entirely right, otherwise LGTM.
I think that was already fixed in d234275, unless there's something I missed. |
Oh I see, the merge conflict was actually on |
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.
Thanks!
Specifies the versions of the action, CLI and GHES that are jointly compatible.
When reviewing, please make sure this is table is clear to read and easy to maintain as we release new versions of the action.
Merge / deployment checklist