Option for no trailing whitespace

jcberquist / commandbox-cfformat

A CommandBox module for formatting CFML component files.

MIT License

22 stars 10 forks source link

Option for no trailing whitespace #127

Closed sbleon closed 3 years ago

sbleon commented 3 years ago

I've found a couple of situations in which cfformat adds trailing whitespace to a line, which git then complains about.

// Some text.
//             <----- This line has one trailing space
// Some more text

<cfsavecontent variable='whatver'>
  This is some text.
            <!----- This line has two leading/trailing spaces
  And here is some more
</cfsavecontent>

I would like an option to eliminate trailing whitespace, because git strongly dislikes trailing whitespace.

jcberquist commented 3 years ago

Thanks! This should be better in v0.16.8.

sbleon commented 3 years ago

Thanks, John!

On Tue, Sep 21, 2021 at 3:45 PM John Berquist @.***> wrote:

Closed #127 https://github.com/jcberquist/commandbox-cfformat/issues/127 via 7821fe0 https://github.com/jcberquist/commandbox-cfformat/commit/7821fe01db430fddcc0d41d46c0914abc5c7008b .

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jcberquist/commandbox-cfformat/issues/127#event-5338284416, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABQQNZKZHQMXAXU5BJHU4LUDDOEPANCNFSM5D6BYM4A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.