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

Removing leading NL, compatible with Python textwrap.dedent("""\. #14

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

marhar
Copy link

@marhar marhar commented May 23, 2022

(sorry for the previous polluted pull request, my git-fu seems to be weak!)

@silverwind
Copy link

silverwind commented May 24, 2022

Agree but I think it should also strip trailing newlines, e.g. perform a strings.TrimSpace, which would match the behaviour of https://github.com/dmnd/dedent.

@lithammer
Copy link
Owner

lithammer commented May 27, 2022

It seems like this is basically a duplicate of #7 (but less efficient).

@lithammer
Copy link
Owner

Not sure why you're messing around with go.mod either?

@marhar
Copy link
Author

marhar commented May 27, 2022

Try it now... I think my mental model is broken, I was thinking it was a self-contained patch, and was fiddling around to understand go.mod. I think I've set everything back to how it should be and won't touch it.

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