pgj commented on PR #5348:
URL: https://github.com/apache/couchdb/pull/5348#issuecomment-2536250640

   > When the first failure happens, it's nicer if it stops the test suite 
right there and makes the failure obvious so we don't have to hunt in the 
console logs for the first of failure. EUnit is a bit annoying in that respect
   
   I do not think that the Mango test suite logs are that elaborate as the ones 
for EUnit.  When there are no failures, `nose2` does not show much other that 
the dots for the progress (a handful of log lines).  Only failures are listed 
which are all relevant for the CI logs in my humble opinion.
   
   > Another positive of `-F` is it should release the CI worker earlier so 
other PR or Full CI runs can executed.
   
   I am surprised: 50-60 seconds mean so much?
   
   > Now locally, I would prefer without `-F` to catch all the failures and fix 
them all at once preferably.
   
   Yeah, it would be nice to make the inclusion of the flag optional.  For 
example, let us remove it by default, and add it for the `MANGO_TEST_OPTS` 
`make` parameter for the CI run?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to