Version: unspecified
Severity: normal
Description
Details
- Reference
- bz47161
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Invalid | None | T51987 UploadWizard has some pretty awful code in places (tracking) | |||
Resolved | Paladox | T62619 All repositories should pass jshint test | |||
Resolved | MarkTraceur | T49161 UploadWizard fails badly with jshint |
Event Timeline
https://github.com/wikimedia/mediawiki-extensions-UploadWizard/pull/1
(Yuvi is going to use this to test his new GH->Gerrit system)
Related URL: https://gerrit.wikimedia.org/r/59064 (Gerrit Change I02772c5c7e570a549b859840ef13f708817661e6)
Related URL: https://gerrit.wikimedia.org/r/59064 (Gerrit Change I02772c5c7e570a549b859840ef13f708817661e6)
Related URL: https://gerrit.wikimedia.org/r/59064 (Gerrit Change I02772c5c7e570a549b859840ef13f708817661e6)
Related URL: https://gerrit.wikimedia.org/r/59064 (Gerrit Change I02772c5c7e570a549b859840ef13f708817661e6)
Related URL: https://gerrit.wikimedia.org/r/59064 (Gerrit Change I02772c5c7e570a549b859840ef13f708817661e6)
Related URL: https://gerrit.wikimedia.org/r/59064 (Gerrit Change I02772c5c7e570a549b859840ef13f708817661e6)
Related URL: https://gerrit.wikimedia.org/r/59064 (Gerrit Change I02772c5c7e570a549b859840ef13f708817661e6)
Don't merge yet, this shall be nice testing grounds for https://github.com/yuvipanda/SuchABot :)
Related URL: https://gerrit.wikimedia.org/r/59108 (Gerrit Change I8bde674a36743592f85e42c0b2e205965c4e9f1b)
Related URL: https://gerrit.wikimedia.org/r/59108 (Gerrit Change I8bde674a36743592f85e42c0b2e205965c4e9f1b)
Related URL: https://gerrit.wikimedia.org/r/59108 (Gerrit Change I8bde674a36743592f85e42c0b2e205965c4e9f1b)
Related URL: https://gerrit.wikimedia.org/r/59108 (Gerrit Change I8bde674a36743592f85e42c0b2e205965c4e9f1b)
Related URL: https://gerrit.wikimedia.org/r/59108 (Gerrit Change I8bde674a36743592f85e42c0b2e205965c4e9f1b)
Related URL: https://gerrit.wikimedia.org/r/59108 (Gerrit Change I8bde674a36743592f85e42c0b2e205965c4e9f1b)
Related URL: https://gerrit.wikimedia.org/r/59108 (Gerrit Change I8bde674a36743592f85e42c0b2e205965c4e9f1b)
Related URL: https://gerrit.wikimedia.org/r/59111 (Gerrit Change I86436fd6128343eff7fe57ff47de409e87a9b946)
Related URL: https://gerrit.wikimedia.org/r/59111 (Gerrit Change I86436fd6128343eff7fe57ff47de409e87a9b946)
Related URL: https://gerrit.wikimedia.org/r/59111 (Gerrit Change I86436fd6128343eff7fe57ff47de409e87a9b946)
Related URL: https://gerrit.wikimedia.org/r/59111 (Gerrit Change I86436fd6128343eff7fe57ff47de409e87a9b946)
Related URL: https://gerrit.wikimedia.org/r/59111 (Gerrit Change I86436fd6128343eff7fe57ff47de409e87a9b946)
Related URL: https://gerrit.wikimedia.org/r/59111 (Gerrit Change I86436fd6128343eff7fe57ff47de409e87a9b946)
Related URL: https://gerrit.wikimedia.org/r/59111 (Gerrit Change I86436fd6128343eff7fe57ff47de409e87a9b946)
It looks like there's still significant amounts of work to do on this - should I leave you assigned, Krenair, or do you want to pass this on to the crowd?
Tooo late. I started a big ol' patch to fix things. I may split it up later if asked nicely.
Sorry, I kept meaning to reply to this but never got around to it. Please 'pass this on to the crowd' :)
Change 80523 had a related patch set uploaded by MarkTraceur:
Loads of work on jshint fixes
Once the patches in gerrit are merged, we can mark this bug as fixed and make the jshint job voting in Jenkins. Exciting!
Change 86776 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.ConfirmCloseWindow.js
Change 86777 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardDeed.js
Change 86778 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardDescription.js
Change 86779 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardDetails.js
Change 80945 abandoned by MarkTraceur:
More jshint fixes
Reason:
Split into:
- If3023606e187afd3ac49739de7545f4fd3ba60f4
- I66a32316613a0779a7ca3a5b77698cacca1ba11f
- Iafb382ca73652b254e097dce7521eee6a783a504
- If7e2ab0fc347841ead3073bc63ba85b4a96795ab
Change 86789 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardUpload.js
Change 86787 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardLicenseInput.js
Change 80947 abandoned by MarkTraceur:
Further jshint fixes
Reason:
Split into:
- I3552fcb0ccd4902c3cd02f2c6eb133b752e3d1ff
- I1c54a2c2f82dcddf8bdfa63ff80b320744aa6b2f
Change 86791 had a related patch set uploaded by MarkTraceur:
Ignore unused jasmine tests for jshint
Change 86792 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardPage.js
Change 86793 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardUploadInterface.js
Change 86794 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UploadWizardUtil.js
Change 86797 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.UtilitiesTime.js
Change 86798 had a related patch set uploaded by MarkTraceur:
jshint fixes for mw.fileApi.js
Change 86793 abandoned by MarkTraceur:
jshint fixes for mw.UploadWizardUploadInterface.js
Reason:
I0710fd5fb9ce8ad21b7d3a45834fa8955944e433 merged
Change 86794 abandoned by MarkTraceur:
jshint fixes for mw.UploadWizardUtil.js
Reason:
I0710fd5fb9ce8ad21b7d3a45834fa8955944e433 merged
Change 86797 abandoned by MarkTraceur:
jshint fixes for mw.UtilitiesTime.js
Reason:
I0710fd5fb9ce8ad21b7d3a45834fa8955944e433 merged
Change 86798 abandoned by MarkTraceur:
jshint fixes for mw.fileApi.js
Reason:
I0710fd5fb9ce8ad21b7d3a45834fa8955944e433 merged
Change 86792 abandoned by MarkTraceur:
jshint fixes for mw.UploadWizardPage.js
Reason:
I0710fd5fb9ce8ad21b7d3a45834fa8955944e433 merged
Change 86791 abandoned by MarkTraceur:
Ignore unused jasmine tests for jshint
Reason:
I0710fd5fb9ce8ad21b7d3a45834fa8955944e433 merged
Sorry about all of the notifications - might have been smart to have split this into multiple bugs...ah well, too late to turn back now!
Ignoring MERGED and ABANDONED patchsets in Gerrit, this is what is left here:
https://gerrit.wikimedia.org/r/#/c/86777/
https://gerrit.wikimedia.org/r/#/c/86779/
https://gerrit.wikimedia.org/r/#/c/86789/
https://gerrit.wikimedia.org/r/#/c/86787/
Change 131686 had a related patch set uploaded by Gilles:
Fix miscellaneous jshint issues
So apparently UploadWizard pass the Jenkins job jslint which runs JSHint. Should we consider this bug fixed and make the job voting? :-)
Change 134791 had a related patch set uploaded by Gilles:
Make jslint voting for UploadWizard
Change 141190 had a related patch set uploaded by Inchikutty:
jshint fixes for mw.UploadWizardLicenseInput.js
Change 141195 had a related patch set uploaded by Inchikutty:
jshint fixes for mw.UploadWizardUpload.js
Change 141198 had a related patch set uploaded by Inchikutty:
Fix miscellaneous jshint issues
Change 141209 had a related patch set uploaded by Inchikutty:
jshint fixes for mw.UploadWizardDeed.js
Change 141190 abandoned by Alex Monk:
jshint fixes for mw.UploadWizardLicenseInput.js
(Someone reuploaded every change since what appeared to be the 26th of Febuary. I have cleaned them up.)