https://bugzilla.wikimedia.org/show_bug.cgi?id=69445

--- Comment #7 from Gilles Dubuc <gdu...@wikimedia.org> ---
The fact that people who aren't developers and/or who don't understand what
custom JS they're deploying to millions of users is the problem. It's not a
situation that needs to be perpetuated through dumbed-down on-wiki code review.
Like MZ just described, if there's a way to remove the bulk of the need for
these users to deploy custom JS, it should be explored first.

And for whatever JS is left that can't be standardized into a global gadget
repository (I haven't looked at the code, but I imagine there will always be
"that specific thing that can't be done differently"), community members with
appropriate skills should be doing it. If you're not willing to learn git,
you're not the right person to be pushing custom JS to a top-5 website. Even if
you don't want to learn or don't have time to, it just means that you need to
enlist the help of someone who knows about this stuff. Right now that step
doesn't exist and irresponsible actions are being taken by people who are just
unaware of what they're doing.

I think that a "simple mode" review tool is also likely to make the uneducated
people sending JS they don't understand less likely to learn to master what
they're doing. As such it wouldn't be code review, it would be "please make
this thing I don't understand work and deploy it", which puts all the workload
burden on the reviewer, instead of being the 2-way street code review should
be.

-- 
You are receiving this mail because:
You are the assignee for the bug.
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to