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

Upgrade the automerge-released-trunk, coverage-report, and prepare-php actions to use Node.js v20 #114

Conversation

eason9487
Copy link
Member

Changes proposed in this Pull Request:

Part of #108

This PR upgrades 3 actions that mainly use external actions:

  • automerge-released-trunk
  • coverage-report
  • prepare-php

Detailed test instructions:

💡 I don't prepare a test for the automerge-released-trunk action as it has only an update for actions/checkout and it should work well.

  1. View a previous workflow run used v1 coverage-report and prepare-php
  2. View a test workflow run used updated coverage-report and prepare-php
  3. View the test PR
    • woocommerce/automatewoo/pull/1740
    • The Coverage report was commented correclty.
      image

@eason9487 eason9487 requested a review from a team April 23, 2024 11:06
@eason9487 eason9487 self-assigned this Apr 23, 2024
@eason9487 eason9487 changed the title Upgrade the automerge-released-trunk, coverage-report, and prepare-php actions in the github-actions package to use Node.js v20 Upgrade the automerge-released-trunk, coverage-report, and prepare-php actions to use Node.js v20 Apr 24, 2024
@eason9487
Copy link
Member Author

I will be skipping the code review process as it can be optional for a devs-facing only change.

Base automatically changed from update/108-nodejs-v20-root-deps to update/108-nodejs-v20 April 25, 2024 02:58
@eason9487 eason9487 merged commit 5e32a7a into update/108-nodejs-v20 Apr 25, 2024
2 checks passed
@eason9487 eason9487 deleted the update/108-nodejs-v20-github-actions-external-only branch April 25, 2024 03:01
@eason9487 eason9487 mentioned this pull request Apr 25, 2024
26 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant