Page MenuHomePhabricator
Create Task
Maniphest T55067

Make sure all JS files have wrapper functions
Closed, ResolvedPublic
Actions

Description

https://www.mediawiki.org/wiki/CC/JS#Closure

We're pretty bad about this currently. Let's do better!

Version: master
Severity: normal

Details

Reference
bz53067

Related Objects
Search...

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:00 AM
• bzimport added a project: UploadWizard.
• bzimport set Reference to bz53067.
• bzimport added a subscriber: Unknown Object (MLST).
MarkTraceur created this task.Aug 19 2013, 7:23 PM
gerritbot added a comment.Aug 20 2013, 7:41 PM

Change 80122 had a related patch set uploaded by MarkTraceur:
Put closures everywhere

https://gerrit.wikimedia.org/r/80122

gerritbot added a comment.Aug 26 2013, 1:51 AM

Change 80122 merged by jenkins-bot:
Enclose every JS file in per-file closures

https://gerrit.wikimedia.org/r/80122

• Gilles added a project: Multimedia.Dec 4 2014, 9:32 AM
• Gilles moved this task from Untriaged to Done on the Multimedia board.Dec 4 2014, 9:34 AM
• Phabricator_maintenance removed a subscriber: yuvipanda.Jun 7 2017, 7:02 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL