[Opendnssec-develop] RE: Regression testing update

Matthijs Mekking matthijs at nlnetlabs.nl
Mon Dec 10 10:00:52 UTC 2012


On 12/06/2012 05:58 PM, Sara Dickinson wrote:
> 
> On 6 Dec 2012, at 14:54, Matthijs Mekking wrote:
>>>> Good point - but that does seem to be how most people work at the moment.... And could always do:
>>>
>>> SUPPORT-XX & TEST: Fix bug and update regression test
>>
>> Yes, but (nit warning) I would prefer
>>
>> SUPPORT-XX
>>
>> TEST: Fix bug
>>
>> (separated by newlines)
> 
> (Nit-tastic warning) Wouldn't:

I asked for it, didn't I? :)

> 
> SUPPORT-XX: Fix bug where.....
> TEST: Added regression test X-Y-Z-01 to cover this scenario
> 
> be even better if you are doing both in one return? However, my main motivation was to make it easy to filter out updates that ONLY affected regression tests so I'm not that bothered. And this is mainly useful when I am looking for possibly important updates that went in without an issue number :-)

My motivation is one update per line for readability.

> 
> We could come up with a whole range of identifiers (BUG: BUILD: RELEASE:) if people thought it was useful?

Let's not overengineer. For me, SUPPORT/OPENDNSSEC/TEST are sufficient
for now.

Best regards,
  Matthijs


> 
> Sara.
> 
> _______________________________________________
> Opendnssec-develop mailing list
> Opendnssec-develop at lists.opendnssec.org
> https://lists.opendnssec.org/mailman/listinfo/opendnssec-develop
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 553 bytes
Desc: OpenPGP digital signature
URL: <http://lists.opendnssec.org/pipermail/opendnssec-develop/attachments/20121210/6ad13257/attachment.bin>


More information about the Opendnssec-develop mailing list