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

Update re_test-integration-prepare.yml #1548

Merged
merged 1 commit into from
Sep 4, 2024
Merged

Update re_test-integration-prepare.yml #1548

merged 1 commit into from
Sep 4, 2024

Conversation

MarcAstr0
Copy link
Collaborator

@MarcAstr0 MarcAstr0 commented Sep 4, 2024

Description

As announced in the GitHub blog, starting September 2nd, the upload-artifact action ignores hidden files and directories by default. Since our integration tests publish our artifacts to a hidden directory (packages/framework-integration-tests/.booster), this change breaks them, so the include-hidden-files option needs to be added to the workflow.

Changes

  • Add include-hidden-files: true option to the upload-artifact action.

Checks

  • Project Builds
  • Project passes tests and checks
  • Updated documentation accordingly

Copy link

what-the-diff bot commented Sep 4, 2024

PR Summary

  • Inclusion of Hidden Files in Testing Workflow
    This update applies a change to the testing workflow setup file (re_test-integration-prepare.yml). Specifically, it introduces a setting (include-hidden-files: true) that allows for hidden files to be included during testing. Previously, these files may have been ignored, but can now be considered in the testing process. This ensures all aspects of the code are examined, providing a more comprehensive check of the code's behavior.

@MarcAstr0 MarcAstr0 merged commit d88ea84 into main Sep 4, 2024
3 checks passed
@MarcAstr0 MarcAstr0 deleted the MarcAstr0-patch-1 branch September 4, 2024 14:56
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