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

#6154 git_status_list_new case insensitive fix #6159

Merged
merged 1 commit into from
Jan 6, 2022

Conversation

arroz
Copy link
Contributor

@arroz arroz commented Jan 1, 2022

Fixes issue #6154.

(Sorry for the re-submission, improved test implementation so no new test repo had to be added).

@ethomson ethomson merged commit ca9f6b9 into libgit2:main Jan 6, 2022
@ethomson
Copy link
Member

ethomson commented Jan 6, 2022

I merged this manually to resolve the conflicts. Thanks for the fix! One thing to keep in mind is that we don't use // style comments in libgit2 - I cleaned those up manually. (I should probably add a linter rule here. 🤔)

@ethomson ethomson added the bug label Feb 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants