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

Change Default Seperator to ;\n #120

Closed
regnerus opened this issue Feb 27, 2015 · 3 comments · Fixed by #122
Closed

Change Default Seperator to ;\n #120

regnerus opened this issue Feb 27, 2015 · 3 comments · Fixed by #122

Comments

@regnerus
Copy link

In the example configuration the default separator is listed as separator: ';' this sometimes leads to issues could this be changed to separator: ';\n'.

@schmod
Copy link

schmod commented Mar 16, 2015

The default separator is grunt.util.linefeed, not ';'.

The example specifically refers to how one would override that default...

That being said, ;\n is indeed a safer example to give for JavaScript, because you could accidentally end up concatenating live code into a // comment.

@arthurvr
Copy link
Contributor

I opened a PR for this. See #122

@schmod
Copy link

schmod commented Mar 26, 2015

Looks good. Thanks!

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 a pull request may close this issue.

3 participants