Skip to content

💚 Attempt to patch untar step #4375

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

Merged
merged 2 commits into from
Oct 23, 2023
Merged

💚 Attempt to patch untar step #4375

merged 2 commits into from
Oct 23, 2023

Conversation

dubzzz
Copy link
Owner

@dubzzz dubzzz commented Oct 23, 2023

CI is currently failing on main branch only. PRs themselves pass well but once they reach main nothing passes anymore.

They start to fail when untaring the generated packages.

This PR tries to do something to avoid the issue. But as it cannot be reproduced except on main, let's see how it will do.

Category:

  • ✨ Introduce new features
  • 📝 Add or update documentation
  • ✅ Add or update tests
  • 🐛 Fix a bug
  • 🏷️ Add or update types
  • ⚡️ Improve performance
  • Other(s): ...

Potential impacts:

  • Generated values
  • Shrink values
  • Performance
  • Typings
  • Other(s): ...

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
CI is currently failing on main branch only. PRs themselves pass well but once they reach main nothing passes anymore.

They start to fail when untaring the generated packages.

This PR tries to do something to avoid the issue. But as it cannot be reproduced except on main, let's see how it will do.
@dubzzz dubzzz enabled auto-merge (squash) October 23, 2023 19:01
@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 23, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 09acbc7:

Sandbox Source
Vanilla Configuration
@fast-check/examples Configuration

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@codecov
Copy link

codecov bot commented Oct 23, 2023

Codecov Report

Merging #4375 (09acbc7) into main (c0c46a8) will not change coverage.
Report is 6 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #4375   +/-   ##
=======================================
  Coverage   93.57%   93.57%           
=======================================
  Files         207      207           
  Lines        5013     5013           
  Branches     1368     1368           
=======================================
  Hits         4691     4691           
  Misses        322      322           
Flag Coverage Δ
unit-tests 93.57% <ø> (ø)
unit-tests-16.x-Linux 93.57% <ø> (ø)
unit-tests-18.x-Linux 93.57% <ø> (ø)
unit-tests-latest-Linux 93.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@dubzzz dubzzz merged commit 994879b into main Oct 23, 2023
@dubzzz dubzzz deleted the fix-ci branch October 23, 2023 20:03
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.

None yet

1 participant