Patch Set 3: (1 comment)
https://gerrit.osmocom.org/#/c/5211/3//COMMIT_MSG Commit Message: Line 7: cosmetic: remove excess whitespaces in Makefile.am > now you've removed the detail I added to the commit summary. to clarify, we have numerous Makefile.am around everywhere. While it doesn't really matter for whitespace, still follow the habit of *pinpointing* what it is about. Here the unambiguous detail would be 'tests/Makefile.am' as I placed in patch set 2. Again I don't want to appear pedantic on cosmetic nonsense, but the point why I'm cracking up on it now is: I don't want to be the commit summary editor for everyone else, it's trivial enough for everyone to get right the first time themselves. It appears I need to be firm to get the point across... Thanks again :) -- To view, visit https://gerrit.osmocom.org/5211 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7090d46ae72183014e6c74d6ddb240af8d66e0b4 Gerrit-PatchSet: 3 Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <[email protected]> Gerrit-HasComments: Yes
