Skip to content

Add --git-dir and --work-tree to user tests #35362

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 1 commit into from
Feb 12, 2020
Merged

Conversation

jablko
Copy link
Contributor

@jablko jablko commented Nov 26, 2019

If you run the user tests with git rebase --exec, Git exports GIT_DIR and GIT_WORK_TREE and ends up running git reset --hard, git clean --force and git pull --force on the TypeScript project, not the test case subproject, regardless of { cwd: submoduleDir } (⌣̩̩́_⌣̩̩̀)

--git-dir and --work-tree override GIT_DIR and GIT_WORK_TREE, forcing it to reset the correct repository.

@jablko jablko force-pushed the patch-31 branch 3 times, most recently from 1036388 to 3ef366d Compare November 29, 2019 22:25
@jablko jablko force-pushed the patch-31 branch 3 times, most recently from 01cf1d7 to e6a39b5 Compare January 25, 2020 20:23
@sandersn sandersn added the Housekeeping Housekeeping PRs label Feb 1, 2020
@jablko jablko force-pushed the patch-31 branch 5 times, most recently from a1fc6f2 to a0a04df Compare February 9, 2020 17:19
@jablko jablko force-pushed the patch-31 branch 3 times, most recently from 3c2bbf7 to 95d170a Compare February 12, 2020 00:49
@sandersn
Copy link
Member

Your command looks like git rebase -x "gulp runtests-parallel --ru=user", right?

Copy link
Member

@sandersn sandersn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me locally.

@sandersn sandersn merged commit 98492d8 into microsoft:master Feb 12, 2020
@jablko
Copy link
Contributor Author

jablko commented Feb 13, 2020

Yes, pretty much. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Housekeeping Housekeeping PRs
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants