Skip to content

Update libgit2 to ef5a385 #1728

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 4 commits into from
Oct 16, 2019
Merged

Update libgit2 to ef5a385 #1728

merged 4 commits into from
Oct 16, 2019

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