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

Fix hidden files not being uploaded with version upgrade in #3576 #3724

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

WorldSEnder
Copy link
Member

Description

It was not prominently explained that "hidden" files, i.e. starting with "." are now ignored with upload-artifact v4.4. This is only a problem when uploading globbed directories in benchmark where we don't know the full path by default.

Checklist

  • I have reviewed my own code
  • I have added tests

it was not prominently explained that "hidden" files, i.e. starting with "." are now ignored
this is only a problem when uploading globbed directories in benchmark where we don't
know the full path by default
@WorldSEnder WorldSEnder added performance A-ci Area: The continuous integration labels Sep 4, 2024
@ranile
Copy link
Member

ranile commented Sep 5, 2024

This should have a better title, or at least better commit message when merging.

@WorldSEnder WorldSEnder changed the title Fixup of #3576 Fix hidden files not being uploaded with version upgrade in #3576 Sep 5, 2024
@WorldSEnder WorldSEnder merged commit 15ac51c into yewstack:master Sep 5, 2024
4 of 5 checks passed
@WorldSEnder WorldSEnder deleted the fixup-download-actionv4 branch September 5, 2024 18:35
@WorldSEnder WorldSEnder mentioned this pull request Sep 14, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Area: The continuous integration performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants