Skip to content

Conversation

@skshetry
Copy link
Collaborator

When we were converting windows path to posixpath, the values
were not in effect because we were not setting it to the
variable.

Also fixes #11, which unskips on Windows + pygit2 backend, which
was due to the above bug.

When we were converting windows path to posixpath, the values
were not in effect because we were not setting it to the
variable.

Also fixes #11, which unskips  on Windows + pygit2 backend, which
was due to the above bug.
@skshetry skshetry requested review from efiop and pmrowla January 10, 2022 09:21
@skshetry skshetry self-assigned this Jan 10, 2022
@skshetry
Copy link
Collaborator Author

Test failures are unrelated, see #27.

Copy link
Contributor

@efiop efiop left a comment

Choose a reason for hiding this comment

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

Great catch! 🔥

@efiop efiop merged commit 6e8941d into main Jan 10, 2022
@efiop efiop deleted the fix-11 branch January 10, 2022 11:37
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.

test_ignored is failing on pygit2 backend on Windows.

2 participants