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

Support preventIndent option #93

Merged
merged 3 commits into from
Apr 29, 2016
Merged

Support preventIndent option #93

merged 3 commits into from
Apr 29, 2016

Conversation

rfunduk
Copy link
Contributor

@rfunduk rfunduk commented Apr 28, 2016

preventIndent can solve a lot of problems with extra whitespace in pre and textarea tags.

See handlebars-lang/handlebars.js#858

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b492630 on rfunduk:prevent-indent into * on pcardune:master*.

@pcardune
Copy link
Owner

Looks good to me! @rfunduk can you update the CHANGELOG.md file? Then I will go ahead and merge + cut a new release.

@rfunduk
Copy link
Contributor Author

rfunduk commented Apr 28, 2016

Yea sure, I wasn't sure if you wanted that since it reads like you wrote the updates. Incoming.

@pcardune
Copy link
Owner

yeah, I need to create a CONTRIBUTING.md doc so these things are more obvious.

@pcardune
Copy link
Owner

Sorry, I just updated the CHANGELOG with another change, so you'll need to rebase.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.473% when pulling ac5efa2 on rfunduk:prevent-indent into bed6096 on pcardune:master.

@pcardune pcardune merged commit eb0cf20 into pcardune:master Apr 29, 2016
@pcardune
Copy link
Owner

Thanks for the contribution! I'll try to make a release today.

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