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

Strip extra trailing new lines #13

Closed
jayvdb opened this issue Nov 19, 2017 · 2 comments
Closed

Strip extra trailing new lines #13

jayvdb opened this issue Nov 19, 2017 · 2 comments

Comments

@jayvdb
Copy link
Member

jayvdb commented Nov 19, 2017

In #4, we kept the trailing blank line, which is good ..

but, it would be nice if after all operations, before the file is written, a final modification is made to ensure there is only one single trailing line feed.
i.e. no blank lines.

Otherwise, we need to remove trailing blank lines like https://github.com/moremoban/setupmobans/blob/78bcfbc/templates/gitignore.jj2#L99

@chfw
Copy link
Member

chfw commented Nov 20, 2017

The right number of Blank lines helps reading. Are you sure that you won't read the files yourself?

@chfw chfw closed this as completed in 8a8a730 Jan 10, 2018
@chfw
Copy link
Member

chfw commented Jan 10, 2018

I misunderstood your requirement but it is done now.

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

No branches or pull requests

2 participants