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

Properly separate message from trailers #38

Merged
merged 1 commit into from
Sep 29, 2023
Merged

Conversation

jonallured
Copy link
Owner

I'm pretty confused about why this used to work but then didn't and now this fixes it? Like if I didn't have enough newlines then shouldn't this have been an issue all along? And if I did have enough newlines at some point then what changed where now I don't?

Mysterous!

/cc @pvinis

@jonallured jonallured self-assigned this Sep 29, 2023
@jonallured jonallured merged commit 9424876 into main Sep 29, 2023
@jonallured jonallured deleted the fix-amend-command branch September 29, 2023 19:25
@pvinis
Copy link

pvinis commented Sep 29, 2023

mysterious indeed. it could be oclif or git or bash, or something else i cant think of. oclif you didnt update i guess? git could be that it changed something. bash, that seems unlikely, since it kind kinda broke for everyone at a similar time.

maybe at one point we will figure it out, or maybe it will stay as another one of the great mysteries of our universe.

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.

2 participants