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

Migrate upload-artifact action to v4 and update cypress dependent packages #1993

Merged

Conversation

daveajrussell
Copy link
Contributor

@daveajrussell daveajrussell commented Feb 4, 2025

V3 of the upload/download artifact actions have been deprecated, and removed since January 25th https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

This PR migrates to v4.

It doesn't seem necessary to make any additional changes outlined in the migration guide https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md here, since it doesn't look like we were previously mutating uploaded artifacts from different runs.

We are also updating some packages that cypress is dependent on in the image, see https://docs.cypress.io/app/get-started/install-cypress#UbuntuDebian

ubuntu-latest resolves to 24.04 https://github.com/actions/runner-images?tab=readme-ov-file#available-images

@daveajrussell daveajrussell requested a review from a team as a code owner February 4, 2025 13:26
@@ -39,13 +39,13 @@ jobs:
- run: yarn install --frozen-lockfile
- run: yarn build
- run: sudo apt-get update
- run: sudo apt-get -y install libgtk2.0-0 libgtk-3-0 libgbm-dev libnotify-dev libgconf-2-4 libnss3 libxss1 libasound2 libxtst6 xauth xvfb
- run: sudo apt-get -y install libgtk2.0-0t64 libgtk-3-0t64 libgbm-dev libnotify-dev libnss3 libxss1 libasound2t64 libxtst6 xauth xvfb
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@daveajrussell daveajrussell changed the title Migrate upload-artifact action to v4 Migrate upload-artifact action to v4 and update cypress dependent packages Feb 4, 2025
@daveajrussell daveajrussell merged commit 7e497b9 into neo4j:master Feb 4, 2025
16 checks passed
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