darcs

Patch 1576 removed failing test for non-issue1363

Title removed failing test for non-issue1363
Superseder Nosy List bf
Related Issues
Status accepted Assigned To
Milestone

Created on 2017-08-11.18:54:56 by bf, last changed 2017-10-12.09:18:02 by bf.

Files
File name Status Uploaded Type Edit Remove
removed-failing-test-for-non_issue1363.dpatch bf, 2017-08-11.18:54:56 text/x-darcs-patch
See mailing list archives for discussion on individual patches.
Messages
msg19551 (view) Author: bf Date: 2017-08-11.18:54:56
See issue1363 for details.
Attachments
msg19738 (view) Author: ganesh Date: 2017-10-12.07:17:25
Accepted as I agree this test is bogus, though as per my comment on 
issue1363 there may be a real issue somewhere.
msg19740 (view) Author: ganesh Date: 2017-10-12.07:30:03
btw it'd be good in general to include a short explanation in the patch 
comment itself, as the patch history may last longer than the bug 
tracker (e.g. "the test is invalid as the conflicts are being reported 
in different contexts".
msg19745 (view) Author: bf Date: 2017-10-12.09:18:02
Yes, should have done that. (If I hadn't screened this patch you could
have amended it to add the longer comment.)
History
Date User Action Args
2017-08-11 18:54:56bfcreate
2017-08-11 19:16:07bfsetstatus: needs-screening -> needs-review
2017-10-12 07:17:25ganeshsetstatus: needs-review -> accepted
messages: + msg19738
2017-10-12 07:30:03ganeshsetmessages: + msg19740
2017-10-12 09:18:02bfsetmessages: + msg19745