Merge lp:~vrruiz/unity-webapps-launchpad/const-to-var into lp:unity-webapps-launchpad

Proposed by Víctor R. Ruiz
Status: Merged
Approved by: Víctor R. Ruiz
Approved revision: 41
Merged at revision: 40
Proposed branch: lp:~vrruiz/unity-webapps-launchpad/const-to-var
Merge into: lp:unity-webapps-launchpad
Diff against target: 22 lines (+4/-2)
1 file modified
Launchpad.user.js (+4/-2)
To merge this branch: bzr merge lp:~vrruiz/unity-webapps-launchpad/const-to-var
Reviewer Review Type Date Requested Status
Alexandre Abreu (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+168769@code.launchpad.net

Commit message

Fix 'const' error (style_checker).

Description of the change

Fix 'const' error (style_checker).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
41. By Víctor R. Ruiz

Fix style_checker error:

Launchpad.user.js:69->Missing space between ';' and '}'.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alexandre Abreu (abreu-alexandre) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Launchpad.user.js'
2--- Launchpad.user.js 2013-05-03 23:12:19 +0000
3+++ Launchpad.user.js 2013-06-12 12:45:31 +0000
4@@ -5,7 +5,7 @@
5 // ==/UserScript==
6
7 // This placeholder gets munged with real data at build time.
8-const WebappsGettextDict = JSON.parse(unescape(
9+var WebappsGettextDict = JSON.parse(unescape(
10 "%7B%22GETTEXT%22%3A%22PLACEHOLDER%22%7D"
11 ));
12
13@@ -66,7 +66,9 @@
14
15 var link = 'https://launchpad.net/' + title;
16 linkVisited(link, title);
17- } catch (x) {linkVisited();}
18+ } catch (x) {
19+ linkVisited();
20+ }
21 addActions();
22 }
23

Subscribers

People subscribed via source and target branches

to all changes: