intelligent-agent / redeem

Firmware for Replicape
http://wiki.thing-printer.com/index.php?title=Redeem
GNU General Public License v3.0
36 stars 44 forks source link

Formatting inconsistency -- yapf tests #175

Open Wackerbarth opened 5 years ago

Wackerbarth commented 5 years ago

I observe that PR#169 includes a trivial change to the last line of particular files that does not affect the parsing of that file.

In one version, the line ends with a newline. In the other, it ends with the end-of-file and no terminating new-line.

Basically, this is the difference in having the newline character be a line separator or having it be the terminal character of each line.

I would prefer that one, or the other, convention be used throughout. In particular, I suggest that we not have a filter that allows both, thus failing to flag trivial changes from one form to the other.