Krinkle has uploaded a new change for review.
https://gerrit.wikimedia.org/r/219300
Change subject: build: Remove 'checkInstallPath' task
......................................................................
build: Remove 'checkInstallPath' task
jsduck doesn't need MW_INSTALL_PATH. Probably a left over?
This is a prerequisite for letting MobileFrontend ride the
npm-run-doc train in Jenkins.
Change-Id: Ic7a89d51c8bfb6e96802d31a79277ca58a2d9182
---
M Gruntfile.js
1 file changed, 1 insertion(+), 23 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/MobileFrontend
refs/changes/00/219300/1
diff --git a/Gruntfile.js b/Gruntfile.js
index a0108b2..31de7bf 100644
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -6,8 +6,6 @@
/*jshint node:true, strict:false*/
module.exports = function ( grunt ) {
- var MW_INSTALL_PATH = grunt.option( 'MW_INSTALL_PATH' ) ||
process.env.MW_INSTALL_PATH;
-
grunt.loadNpmTasks( 'grunt-mkdir' );
grunt.loadNpmTasks( 'grunt-contrib-jshint' );
grunt.loadNpmTasks( 'grunt-jscs' );
@@ -149,12 +147,8 @@
}
} );
- grunt.registerTask( 'checkInstallPath', 'Check if the install path is
set', function () {
- checkInstallPathNotFound( MW_INSTALL_PATH, grunt );
- } );
-
grunt.registerTask( 'lint', [ 'jshint', 'jscs' ] );
- grunt.registerTask( 'docs', [ 'checkInstallPath', 'clean:jsdocs',
'mkdir:jsdocs', 'jsduck:main' ] );
+ grunt.registerTask( 'docs', [ 'clean:jsdocs', 'mkdir:jsdocs',
'jsduck:main' ] );
// grunt test will be run by npm test which will be run by Jenkins
// Do not execute qunit here, or other tasks that require full mediawiki
@@ -163,19 +157,3 @@
grunt.registerTask( 'default', [ 'test' ] );
};
-
-/**
- * Checks if the path is set, and if not it prints an error message and bails
- * out
- */
-function checkInstallPathNotFound( MW_INSTALL_PATH, grunt ) {
- if ( !MW_INSTALL_PATH ) {
- grunt.log.error(
- 'MW_INSTALL_PATH is not set. Please set it to your root
mediawiki installation or pass the --MW_INSTALL_PATH to grunt.\n\n' +
- '\n export
MW_INSTALL_PATH=/Users/johndoe/dev/mediawiki' +
- '\n MW_INSTALL_PATH=../../ grunt' +
- '\n grunt --MW_INSTALL_PATH=../../'
- );
- process.exit( 1 );
- }
-}
--
To view, visit https://gerrit.wikimedia.org/r/219300
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic7a89d51c8bfb6e96802d31a79277ca58a2d9182
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/MobileFrontend
Gerrit-Branch: master
Gerrit-Owner: Krinkle <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
