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 GitHub actions #171

Merged
merged 2 commits into from
Nov 6, 2024
Merged

Upgrade GitHub actions #171

merged 2 commits into from
Nov 6, 2024

Conversation

ianroberts
Copy link
Member

  • upgrade upload-artifact action to v4, since v3 will stop working soon
    • also upgrade upload-pages-artifact, which has a transitive dependency on upload-artifact
  • upgrade checkout and github-script actions to their latest releases.

Also upgraded upload-pages-artifact, as it has a transitive dependency on upload-artifact, and deploy-pages to work with the upgraded upload-pages-artifact
@ianroberts ianroberts marked this pull request as draft November 6, 2024 13:08
@ianroberts
Copy link
Member Author

Draft for now as I'll need to fix up gate-top first.

Copy link

github-actions bot commented Nov 6, 2024

Test Results

108 tests  ±0   108 ✅ ±0   26s ⏱️ +19s
 27 suites ±0     0 💤 ±0 
 27 files   ±0     0 ❌ ±0 

Results for commit 1880b7b. ± Comparison against base commit 8ecf711.

♻️ This comment has been updated with latest results.

@ianroberts ianroberts force-pushed the actions/upload-artifact branch from f244daf to 1880b7b Compare November 6, 2024 13:22
@ianroberts ianroberts marked this pull request as ready for review November 6, 2024 13:24
@ianroberts
Copy link
Member Author

Ready for review, but please merge GateNLP/gate-top#22 before this one.

@greenwoodma greenwoodma merged commit 26b10b8 into master Nov 6, 2024
3 checks passed
@greenwoodma greenwoodma deleted the actions/upload-artifact branch November 6, 2024 13:39
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