[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #4 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 120752 had a related patch set uploaded by Prtksxna:
Add .jshintrc file

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

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

Gerrit Notification Bot gerritad...@wikimedia.org changed:

   What|Removed |Added

 Status|NEW |PATCH_TO_REVIEW

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #5 from Antoine hashar Musso has...@free.fr ---
The jslint job is only triggered when Javascript related files are edited
(.jshint*, *.js or *.json iirc).

Requiring an empty .jshintrc file is bug
https://bugzilla.wikimedia.org/show_bug.cgi?id=52456 :-D  Should be fixed soon
by making sure jshint always uses the defaults.

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #6 from Prateek Saxena psax...@wikimedia.org ---
I've added a non-empty .jshintrc file. Once this patch gets merged we should be
able to close this bug.

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #7 from Gerrit Notification Bot gerritad...@wikimedia.org ---
Change 120752 merged by jenkins-bot:
Add .jshintrc file and update core.js to remove errors

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

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #8 from Prateek Saxena psax...@wikimedia.org ---
https://integration.wikimedia.org/ci/job/mwext-Popups-jslint/12/console

All good, eh? We close this, eh?

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-25 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

Siebrand Mazeland siebr...@kitano.nl changed:

   What|Removed |Added

 Status|PATCH_TO_REVIEW |RESOLVED
 CC||siebr...@kitano.nl
 Resolution|--- |FIXED

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #1 from Antoine hashar Musso has...@free.fr ---
Done with change id
https://gerrit.wikimedia.org/r/#/q/If385e647b8b464b599667d00d6a4b6a0803711a1,n,z


JShint is failing though
https://integration.wikimedia.org/ci/job/mwext-Popups-jslint/1/console

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #2 from Prateek Saxena psax...@wikimedia.org ---
I just pushed some code, which didn't really fix any of those errors and its
not failing -
https://integration.wikimedia.org/ci/job/mwext-Popups-lint/2/console

Does this mean we can close this bug?

-- 
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


[Bug 62997] Add Jenkins jobs for Popups extension repo

2014-03-24 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=62997

--- Comment #3 from Prateek Saxena psax...@wikimedia.org ---
Its happening again
https://integration.wikimedia.org/ci/job/mwext-Popups-jslint/7/console

These errors usually go away after a `recheck` do I need to edit a .jshint
file?

-- 
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