Merge pull request #19 from stephenmathieson/default-to-master-branch

index: Default to `master` when there are no tags
duojs · Feb 1, 2015 · 5e7a416 · 5e7a416
2 parents 3d5aa04 + 531e866
commit 5e7a416
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -111,10 +111,13 @@ function remote(name, token) {
  */
 
 function satisfy(refs, version){
+  var master;
   for (var i = 0, ref; ref = refs[i++];) {
+    if ('master' == ref.name) master = ref;
     if (equal(ref.name, version)) return ref;
   }
-};
+  return master;
+}
 
 /**
  * Arrange the refs

diff --git a/test/gh-resolve.js b/test/gh-resolve.js
@@ -64,6 +64,14 @@ describe('resolve()', function(){
     });
   })
 
+  it('should use master when there are no tags', function(done){
+    resolve('mnmly/slider', function(err, ref){
+      if (err) return done(err);
+      assert('master' == ref.name);
+      done();
+    })
+  })
+
   it('should provide better errors for invalid repos', function(done) {
     resolve('sweet/repo@amazing/version', function(err, ref){
       assert(err);