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

Add support for git format-patch headers #7

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

kit-ty-kate
Copy link
Collaborator

@kit-ty-kate kit-ty-kate commented Mar 20, 2024

Fixes #6

This hardens the parser by allowing to ignore

---

contained in the header of patches outputted by git format-patch

@hannesm
Copy link
Owner

hannesm commented Mar 21, 2024

excellent, thanks! I'll embed the patch file as test case to avoid regressions.

@hannesm hannesm merged commit f35adfe into hannesm:main Mar 21, 2024
1 check passed
hannesm added a commit to hannesm/opam-repository that referenced this pull request Apr 3, 2024
CHANGES:

* Add support for git format-patch headers (hannesm/patch#7 @kit-ty-kate)
* Pretty-printer: fix no_newline support (hannesm/patch#11 @kit-ty-kate)
* Various fixes to the diff parser ('---' mid diff, hunks, no newline at end
  of file) (hannesm/patch#10 @kit-ty-kate)
* Add Patch.pp_list (hannesm/patch#13 @kit-ty-kate)
* Merge Edit and Rename operations (hannesm/patch#14 @kit-ty-kate)
* Add a diff implementation (hannesm/patch#12 @kit-ty-kate)
* Rename to_diffs to parse (hannesm/patch#16 @kit-ty-kate)
* Provide API docs, tweak documentation (@hannesm)
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.

Feature request: be able to parse git patches
2 participants