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

Update major versions of Actions in README and workflows #1012

Merged
merged 4 commits into from Mar 30, 2022

Conversation

henrymercer
Copy link
Contributor

@henrymercer henrymercer commented Mar 30, 2022

Update the README to use the latest major versions of actions/checkout and the Actions in this repo. Also update our workflows to use the latest major versions of various first-party Actions.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • Confirm the changelog has been updated if necessary.

@henrymercer henrymercer requested a review from a team as a code owner Mar 30, 2022
cklin
cklin approved these changes Mar 30, 2022
@henrymercer henrymercer enabled auto-merge Mar 30, 2022
@cklin
Copy link
Contributor

cklin commented Mar 30, 2022

Thank you for the change!

@henrymercer
A question just popped up in my mind. Do we want these documentation changes in the v1 branch as well? I can see arguments both ways...

@henrymercer henrymercer merged commit b0ddf36 into main Mar 30, 2022
243 checks passed
@henrymercer henrymercer deleted the henrymercer/update-actions-major-versions branch Mar 30, 2022
@henrymercer
Copy link
Contributor Author

henrymercer commented Mar 31, 2022

Thank you for the change!

@henrymercer A question just popped up in my mind. Do we want these documentation changes in the v1 branch as well? I can see arguments both ways...

Good question. For now I don’t think I see a good enough reason for the branches to diverge. But very happy to reconsider.

This was referenced Apr 5, 2022
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