Skip to content

PHPC-2365: Update GitHub actions to use Node 20 #1528

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

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

jmikola
Copy link
Member

@jmikola jmikola commented Mar 19, 2024

@jmikola jmikola changed the base branch from master to v1.17 March 19, 2024 20:08
@@ -67,7 +67,7 @@ jobs:

- name: Upload DLL and PDB files as build artifacts
if: ${{ github.event_name == 'push' }}
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only action that had notable changes between v3 and v4 (see: MIGRATION.md, but I don't believe any of those changes applied to our usage.

@jmikola jmikola requested a review from alcaeus March 20, 2024 19:33
@jmikola
Copy link
Member Author

jmikola commented Mar 21, 2024

Windows tests are all failing to start the MongoDB service as usual, but I'm going to proceed with merging.

The logs attest that actions/cache succeeds for saving and restoring.

actions/upload-artifact isn't being run for the PR but I expect it will after merging, and we won't know about dawidd6/action-download-artifact until the next release is triggered.

@jmikola jmikola merged commit 0cdfe3e into mongodb:v1.17 Mar 21, 2024
@jmikola jmikola deleted the 1.17-phpc-2365 branch March 21, 2024 13:46
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.

2 participants