search

threedaymonk / htmlbeautifier

A normaliser/beautifier for HTML that also understands embedded Ruby. Ideal for tidying up Rails templates.
MIT License
325 stars 59 forks source link

allowing optional comment start and end points. They can now be used to... #27

Closed olsonpm closed 9 years ago

olsonpm commented 9 years ago

... remove unwanted whitespace between elements

threedaymonk commented 9 years ago

Thanks for this. However:

  1. Could you please add some tests that explain and cover this change?
  2. Please make sure that your commit messages conform to these guidelines.
olsonpm commented 9 years ago

Will do. Heading to bed for now. Will get to it tomorrow.

On Tuesday, April 21, 2015, Paul Battley notifications@github.com wrote:

Thanks for this. However:

  1. Could you please add some tests that explain and cover this change?
  2. Please make sure that your commit messages conform to these guidelines http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html.

— Reply to this email directly or view it on GitHub https://github.com/threedaymonk/htmlbeautifier/pull/27#issuecomment-94789721 .