-
-
Notifications
You must be signed in to change notification settings - Fork 505
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
Bump workflows #2566
Bump workflows #2566
Conversation
9d2cc87
to
9e9d6dd
Compare
@@ -8,7 +8,7 @@ on: | |||
jobs: | |||
release: | |||
name: "Git tag, release & create merge-up PR" | |||
uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@1.4.1" | |||
uses: "doctrine/.github/.github/workflows/release-on-milestone-closed.yml@3.1.0" |
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.
Anything we need to do bumping two majors?
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.
Apparently not:
2.0: https://github.com/doctrine/.github/releases/tag/2.0.0 was to set PHP 8.1 by default
3.0: https://github.com/doctrine/.github/releases/tag/3.0.0 was to set PHP 8.2 by default and run on ubuntu 22.04
Following ORM, here updated to 2.0 and here to 3.0
Looking at this, we can use ubuntu 22.04 as well, I'll add those changes in this PR
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 for checking!
9993147
to
495ca6b
Compare
495ca6b
to
da55343
Compare
Thanks @franmomu! |
Thanks @franmomu! For the record, dependabot can automatically update GitHub Actions Workflows, and we might want to leverage this here as well. See mongodb/mongo-php-library#1157 for an example :) |
No description provided.