On Mar 30, 2012, at 10:14 AM, Amir E. Aharoni wrote:
Hi,
I made a little localization fix to the jQuery.ui datepicker, which is used by the Upload Wizard. I submitted it upstream through GitHub and it was merged there.
Krinkle says that jQuery is supposed to be only modified upstream, and that is a Good Thing. What is our policy for actually merging upstream jQuery changes to MW code?
I wouldn't go as far as calling it a policy, but I'd recommend we don't do merging of any kind with upstream libraries.
Only update to official (minor or major) releases.
So next time they release, we update the copy in master and from there we make sure things are still compatible and the unit tests pass.
If they consider it an important fix, they'll make a minor release soon, and else we'll have wait for them to release.
If they refuse to release (or if the maintainer isn't active anymore), then we could consider forking it entirely and merging our proposed upstream fixed to master ahead of time (like we did with the jQuery Tipsy plugin), but fortunately that isn't a concern for jQuery UI :)
-- Krinkle