jenkins-bot has submitted this change and it was merged.

Change subject: Rename intro.js and outro.js to have .txt extensions
......................................................................


Rename intro.js and outro.js to have .txt extensions

This will make it easier to e.g. pass all of src/ to jsduck
without it tripping on parse errors.

This also fixes the static jsduck test in Jenkins since we can't
use /dist/ without building a distribution from source first,
and we can't easily pass all files except intro/outro to jsduck
because it doesn't support wildcards and doesn't support excluding
files (we'd have to list each directory and file separately).

Change-Id: I7481ddb210c963063a9483ddc8b3ac1fc0de056e
---
M .jshintignore
M Gruntfile.js
M build/modules.json
M jsduck.config.json
R src/intro.js.txt
R src/outro.js.txt
6 files changed, 4 insertions(+), 8 deletions(-)

Approvals:
  Jforrester: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/.jshintignore b/.jshintignore
index a20a828..01792af 100644
--- a/.jshintignore
+++ b/.jshintignore
@@ -3,5 +3,3 @@
 docs/
 lib/
 node_modules
-src/intro.js
-src/outro.js
diff --git a/Gruntfile.js b/Gruntfile.js
index 3f9480a..e646c2d 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -82,9 +82,7 @@
                jscs: {
                        dev: [
                                '<%= jshint.dev %>',
-                               '!demos/{dist,lib}/**',
-                               '!src/intro.js',
-                               '!src/outro.js'
+                               '!demos/{dist,lib}/**'
                        ],
                        dist: '<%= jshint.dist %>'
                },
diff --git a/build/modules.json b/build/modules.json
index 5942490..9a65934 100644
--- a/build/modules.json
+++ b/build/modules.json
@@ -1,7 +1,7 @@
 {
        "oojs-ui": {
                "scripts": [
-                       "src/intro.js",
+                       "src/intro.js.txt",
                        "src/core.js",
                        "src/Element.js",
                        "src/Frame.js",
@@ -65,7 +65,7 @@
                        "src/widgets/ToggleWidget.js",
                        "src/widgets/ToggleButtonWidget.js",
                        "src/widgets/ToggleSwitchWidget.js",
-                       "src/outro.js"
+                       "src/outro.js.txt"
                ],
                "styles": [
                        "src/styles/intro.css",
diff --git a/jsduck.config.json b/jsduck.config.json
index d36e398..da6cff6 100644
--- a/jsduck.config.json
+++ b/jsduck.config.json
@@ -10,6 +10,6 @@
        "--": [
                "jsduck.external.js",
                "lib/oojs.js",
-               "dist/oojs-ui.js"
+               "src"
        ]
 }
diff --git a/src/intro.js b/src/intro.js.txt
similarity index 100%
rename from src/intro.js
rename to src/intro.js.txt
diff --git a/src/outro.js b/src/outro.js.txt
similarity index 100%
rename from src/outro.js
rename to src/outro.js.txt

-- 
To view, visit https://gerrit.wikimedia.org/r/130623
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7481ddb210c963063a9483ddc8b3ac1fc0de056e
Gerrit-PatchSet: 3
Gerrit-Project: oojs/ui
Gerrit-Branch: master
Gerrit-Owner: Krinkle <krinklem...@gmail.com>
Gerrit-Reviewer: Jforrester <jforres...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to