Merge pull request #14 from dollarshaveclub/fix-glimmer-2-logic

[fix] Correct logic for pushing app-lt-2-10 tree
ember-fastboot · Nov 10, 2016 · a2693bf · a2693bf
2 parents a01b6a1 + e5e1a8f
commit a2693bf
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/config/ember-try.js b/config/ember-try.js
@@ -5,6 +5,24 @@ module.exports = {
       name: 'default',
       dependencies: { }
     },
+    {
+      name: 'ember 2.9.0-beta.5',
+      dependencies: {
+        'ember': '2.9.0-beta.5'
+      }
+    },
+    {
+      name: 'ember 2.9.0',
+      dependencies: {
+        'ember': '2.9.0'
+      }
+    },
+    {
+      name: 'ember 2.10.0-beta.1',
+      dependencies: {
+        'ember': '2.10.0-beta.1'
+      }
+    },
     {
       name: 'ember-release',
       dependencies: {

diff --git a/index.js b/index.js
@@ -17,7 +17,7 @@ module.exports = {
 
     // 2.9.0-beta.1 - 2.9.0-beta.5 used glimmer2 (but 2.9.0 did not)
     // 2.10.0-beta.1+ includes glimmer2
-    if ((emberVersion.gt('2.9.0-beta') && emberVersion.lt('2.9.0')) || emberVersion.gt('2.10.0-beta')) {
+    if (!(emberVersion.gt('2.9.0-beta') && emberVersion.lt('2.9.0')) && !emberVersion.gt('2.10.0-beta')) {
       trees.push(this.treeGenerator(path.resolve(this.root, 'app-lt-2-10')));
     }