Re: [Tails-dev] Adding "look at test suite results" to our r…

Delete this message

Reply to this message
Author: intrigeri
Date:  
To: The Tails public development discussion list
Subject: Re: [Tails-dev] Adding "look at test suite results" to our reviewing guidelines
anonym:
> IMHO those of us with access to jenkins should also make sure to
> check the tests before asking for a review. Clearly I failed at that
> this time (but I have done it at other times, so it was just a slip!),
> so the blame is on me, really! :S


Feel free to add this to our branch submission checklist :)

Still, between a branch submission and when it's reviewed, things may
change and break, so IMO the burden of checking test suite results
should primarily lie on the gatekeeper, that is the person who makes
the decision of merging or not.