www.digitalmars.com         C & C++   DMDScript  

digitalmars.D - [spec] Small or large updates

reply Dibyendu Majumdar <d.majumdar gmail.com> writes:
Hi,

I would prefer to submit small pull requests that do not take 
much time to review and can be merged quickly - and this way work 
through the spec. Is this okay or is it better to make large pull 
requests with a whole bunch of changes?

Thanks
May 14
next sibling parent Nicholas Wilson <iamthewilsonator hotmail.com> writes:
On Tuesday, 14 May 2019 at 21:27:23 UTC, Dibyendu Majumdar wrote:
 Hi,

 I would prefer to submit small pull requests that do not take 
 much time to review and can be merged quickly - and this way 
 work through the spec. Is this okay or is it better to make 
 large pull requests with a whole bunch of changes?

 Thanks
As a reviewer I much prefer smaller PRs, not so much because they are easier to review (I mean its nice, don't get me wrong) but the error messages you get if you muck something up are much easier to pinpoint in smaller PRs. You can get most of the benefits of larger PRs from telling the reviewers what you're woking on an what they can expect to come next. If for some reason that doesn't work submitting multiple PRs that split the changes is also a good option. Thanks Nic
May 14
prev sibling parent Andrei Alexandrescu <SeeWebsiteForEmail erdani.org> writes:
On 5/14/19 10:27 PM, Dibyendu Majumdar wrote:
 Hi,
 
 I would prefer to submit small pull requests that do not take much time 
 to review and can be merged quickly - and this way work through the 
 spec. Is this okay or is it better to make large pull requests with a 
 whole bunch of changes?
Small and meaningful is beautiful.
May 14