Skip to content

Conversation

ethomson
Copy link
Member

Although libgit2 hasn't yet started locking down for its next release, it should be 🔜 . We can start readying our prereleases to match.

@ethomson ethomson force-pushed the ethomson/update_libgit2 branch 2 times, most recently from 886b74a to 8fc106c Compare October 16, 2019 11:01
URLs without a path are perfectly valid; the path is implicitly `/`.
libgit2 was incorrectly rejecting these paths, and LibGit2Sharp was
incorrectly testing this bad behavior.  Remove this test now that
libgit2 has fixed this issue.
@ethomson ethomson force-pushed the ethomson/update_libgit2 branch from 18416a9 to 64ab971 Compare October 16, 2019 16:41
@ethomson
Copy link
Member Author

@bording, can I request a quick review so that I can ship a prerelease nuget and unblock @AArnott? Thanks!

Copy link
Member

@bording bording left a comment

Choose a reason for hiding this comment

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

LGTM

@ethomson ethomson merged commit e237c8b into master Oct 16, 2019
@ethomson ethomson deleted the ethomson/update_libgit2 branch October 16, 2019 18:19
@AArnott
Copy link
Contributor

AArnott commented Oct 22, 2019

Thank you both!

ghost referenced this pull request in mendix/libgit2sharp Dec 15, 2020
ghost referenced this pull request in mendix/libgit2sharp Dec 16, 2020
* Cherry-pick Merge pull request #1728 from libgit2/ethomson/update_libgit2

Co-authored-by: Edward Thomson <ethomson@edwardthomson.com>
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.

3 participants