Purge trailing whitespace #23

Merged
merged 2 commits into from Nov 11, 2016

Conversation

Projects
None yet
4 participants
@paultcochrane
Contributor

paultcochrane commented Nov 10, 2016

Some projects consider this a must, and will disallow commits to be submitted which contain trailing whitespace (the Linux kernel is an example project where trailing whitespace isn’t permitted). Other projects see whitespace cleanup as simply nit-picking. Either way, I thought I'd tidy this up, and if you don't think this is necessary, I don't mind :-). I've split the commits just in case you want to cherry pick the changes.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling eb341df on paultcochrane:pr/purge-trailing-whitespace into ecffd69 on fayland:master.

coveralls commented Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling eb341df on paultcochrane:pr/purge-trailing-whitespace into ecffd69 on fayland:master.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling eb341df on paultcochrane:pr/purge-trailing-whitespace into ecffd69 on fayland:master.

coveralls commented Nov 10, 2016

Coverage Status

Coverage remained the same at 99.492% when pulling eb341df on paultcochrane:pr/purge-trailing-whitespace into ecffd69 on fayland:master.

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Nov 10, 2016

Current coverage is 93.40% (diff: 100%)

Merging #23 into master will not change coverage

@@             master        #23   diff @@
==========================================
  Files            14         14          
  Lines           197        197          
  Methods           0          0          
  Messages          0          0          
  Branches         26         26          
==========================================
  Hits            184        184          
  Misses            1          1          
  Partials         12         12          

Powered by Codecov. Last update ecffd69...eb341df

codecov-io commented Nov 10, 2016

Current coverage is 93.40% (diff: 100%)

Merging #23 into master will not change coverage

@@             master        #23   diff @@
==========================================
  Files            14         14          
  Lines           197        197          
  Methods           0          0          
  Messages          0          0          
  Branches         26         26          
==========================================
  Hits            184        184          
  Misses            1          1          
  Partials         12         12          

Powered by Codecov. Last update ecffd69...eb341df

@fayland fayland merged commit 7d29212 into fayland:master Nov 11, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment