[BUGFIX] add repo back to manifest

brousalis · Jul 17, 2016 · 014b5cc · 014b5cc
1 parent c045ff0
commit 014b5cc
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 1 deletion.
diff --git a/gulpfile.js/lib/manifest.js b/gulpfile.js/lib/manifest.js
@@ -2,6 +2,13 @@
 
 var fs = require('fs');
 
+function repo() {
+  var manifest = exports.file();
+  var repo = manifest.repository && /git@github\.com:([\w-]+)\/([\w-]+)\.git/.exec(manifest.repository.url);
+  if (!repo) repo = /git\:\/\/github\.com\/([\w-]+)\/([\w-]+)\.git/.exec(manifest.repository.url);
+  return repo;
+}
+
 exports.file = function() {
   return JSON.parse(fs.readFileSync('./package.json', 'utf8'));
 }
@@ -13,3 +20,11 @@ exports.version = function() {
 exports.name = function() {
   return exports.file().name
 }
+
+exports.owner = function() {
+  return repo()[1];
+};
+
+exports.repo = function() {
+  return repo()[2];
+};
diff --git a/gulpfile.js/tasks/release.js b/gulpfile.js/tasks/release.js
@@ -98,6 +98,11 @@ function release(callback) {
     return;
   }
 
+  if (!manifest.repo()) {
+    util.errorHandler('release')(new Error('Your package.json is missing the repository field.'));
+    return;
+  }
+
   runSequence(
     'changelog',
     'bump',

diff --git a/gulpfile.js/tasks/test.js b/gulpfile.js/tasks/test.js
@@ -33,7 +33,7 @@ gulp.task('tdd', function (callback) {
 gulp.task('webdriver_update', webdriver_update);
 gulp.task('webdriver_standalone', webdriver_standalone);
 
-gulp.task('e2e', function(callback) {
+gulp.task('e2e', ['webdriver_update'], function(callback) {
   gulp.src(path.join(config.paths.tests, config.paths.e2e, '/**/*.{' + config.extensions.scripts + '}'))
     .pipe(protractor({
       configFile: path.resolve('protractor.conf.js'),