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

tests: Use git worktree correctly #2117

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NoahGorny
Copy link
Member

git worktree -d does not work on all versions of git worktree

this seems to resolve my problems running the tests locally

Description

After debugging the tests_setup a bit, I found out that git worktree call failed. After further testing, it seems like the -d flag was responsible for the failures

Motivation and Context

Closes #2115

How Has This Been Tested?

By running the tests :)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

git worktree -d does not work on all versions of git worktree
@NoahGorny
Copy link
Member Author

nope, seems like there is something else in play here

@tbhaxor
Copy link
Contributor

tbhaxor commented Mar 15, 2022

nope, seems like there is something else in play here

Hey, @NoahGorny I tried testing on Monterey and didn't get any errors. Could it be specific to big sur?

@tbhaxor tbhaxor mentioned this pull request Mar 23, 2022
8 tasks
@georgettica
Copy link
Contributor

yes, I have big sur and it happens to me locally aswell. not sure how to debug && fix

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.

Flaky test skips
3 participants