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

chore: reduce windows shim test flakes #7386

Merged
merged 19 commits into from
Apr 19, 2024

Conversation

lukekarrys
Copy link
Contributor

@lukekarrys lukekarrys commented Apr 18, 2024

  • Uses rimraf.moveRemove to delete node.exe since this is the least error-prone way to make sure it doesn't throw when tap is cleaning it up
  • Always use shell: true when running spawnSync

Not merging this until I get 3 consecutive successful CI runs

@lukekarrys lukekarrys force-pushed the lk/windooooooooooooooooooooooows branch from 22e7b69 to d736700 Compare April 18, 2024 20:41
@lukekarrys lukekarrys force-pushed the lk/windooooooooooooooooooooooows branch from d736700 to 0e5be1a Compare April 18, 2024 20:42
@lukekarrys lukekarrys marked this pull request as ready for review April 18, 2024 23:55
@lukekarrys lukekarrys requested a review from a team as a code owner April 18, 2024 23:55
@lukekarrys lukekarrys changed the title chore: try to reduce windows shim test flakes chore: reduce windows shim test flakes Apr 18, 2024
Co-authored-by: Gar <gar+gh@danger.computer>
@lukekarrys lukekarrys merged commit 9333e9d into latest Apr 19, 2024
18 checks passed
@lukekarrys lukekarrys deleted the lk/windooooooooooooooooooooooows branch April 19, 2024 01:11
@github-actions github-actions bot mentioned this pull request Apr 19, 2024
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