Skip to content

Conversation

itsaphel
Copy link
Contributor

@itsaphel itsaphel commented Feb 1, 2025

The docs job seems to be erroring due to the deprecation of actions/upload-artifact@v3: https://github.com/braindecode/braindecode/actions/runs/13089486769/job/36524496412?pr=690

This PR bumps it to v4. I reviewed the migration guide here: I think it should be fine, but I don't know for sure what the value of GITHUB_ARTIFACT_URL is in CircleCI. I've checked the archive of the last successful docs build - it doesn't look like there's any hidden files of concern.

This PR will need a no changelog label

@itsaphel itsaphel changed the title bump actions/upload-artifact to v4 Bump actions/upload-artifact to v4 Feb 1, 2025
@bruAristimunha bruAristimunha marked this pull request as ready for review February 1, 2025 15:01
@bruAristimunha bruAristimunha merged commit 51a2a3d into braindecode:master Feb 1, 2025
13 of 14 checks passed
@bruAristimunha
Copy link
Collaborator

Thank you @itsaphel

@itsaphel itsaphel deleted the update-docs-github-workflow branch February 1, 2025 15:05
Copy link

codecov bot commented Feb 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.23%. Comparing base (cd8eec2) to head (902bbe7).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #691   +/-   ##
=======================================
  Coverage   87.23%   87.23%           
=======================================
  Files          76       76           
  Lines        6821     6821           
=======================================
  Hits         5950     5950           
  Misses        871      871           

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