[MERGE] Don't raise KnownFailure for all coding_style regressions

Marius Kruger amanic at gmail.com
Mon Mar 30 05:43:22 BST 2009


* Don't raise KnownFailure for all coding_style regressions, but be less strict.
* Just print the number of files with trailing white-space in stead of
failing (like the long-line check).
  I timed the test with and without this and I think the difference is
small enough to leave it in.
    ==with trailing white-space & long-line checks==
    Ran 1 test in 0.507s
    real    0m3.136s
    user    0m2.990s
    sys     0m0.130s
    ==without==
    Ran 1 test in 0.366s
    real    0m3.056s
    user    0m2.930s
    sys     0m0.130s
* I updated the white space policy in HACKING to:
  Trailing white space should be avoided, but is allowed.
  You should however not make lots of unrelated white space changes.

--
Marius
# I only got round to this now because I've been a bit busy at work
and over the weekends
# and my internet at home has been down for most of march and still is :(
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 20090328_1211-bzr.remove_trailing_whitespace.patch
Type: text/x-patch
Size: 4995 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090330/1a25391d/attachment.bin 


More information about the bazaar mailing list