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.

Sign up for GitHub

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

Jump to bottom

_include_mskip_file: push only pure lines (no multiple "\n") #18

Open

dolmen wants to merge 1 commit into master from fix/_include_mskip_file-push-pure-lines

Member

dolmen commented May 28, 2015

sub _include_mskip_file returns an array of strings that are supposed to be lines.
But some of them aren't:

"\n#!start included $mskip\n"
"#!end included $mskip\n\n"

karenetheridge added the bug label


          _include_mskip_file: push only pure lines (no multiple "\n")

f2a9267

Don't push strings with multiple "\n"

dolmen force-pushed the fix/_include_mskip_file-push-pure-lines branch from 3d17e6b to f2a9267 Compare

January 17, 2017 16:31

Member Author

dolmen commented Apr 12, 2017

OK for merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug