Le 18/06/2015 16:18, Brad Jorsch (Anomie) a écrit :
> On Thu, Jun 18, 2015 at 9:32 AM, Jon Robson <jdlrob...@gmail.com> wrote:
>> Smaller commits generally are better
> 
> 
> I'm going to call red herring here. Whether or not smaller commits are
> really better, one patch that does
> 
> + if ( detectOldInput( $input ) ) {
> +     $input = upgradeInput( $input );
> +     // wfDeprecated( "You used the old input style!" );
> + }
>
> then a followup that does
> 
> -     // wfDeprecated( "You used the old input style!" );
> +     wfDeprecated( "You used the old input style!" );
> 
> to be merged 1 second later[1] isn't an example.

Though the second patch should have much more content:
- potentially an announcement (think about MW api.php deprecations)
- migration documentation
- a release note entry

And so on.  So the first patch let devs catch up, the second one is
preparing the proper release/deprecation of the feature.


>  [1]: Assuming Jenkins gets a whole lot faster about merging stuff ;)

Time to get MediaWiki test suite faster :-]  It needs a lot of
refactoring to stop exercising the whole stack for every single "unit"
test :-D


-- 
Antoine "hashar" Musso


_______________________________________________
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Reply via email to