nebius/SWE-rebench-V2-PRs huggingface dataset:
Repository: ember-cli/ember-cli
Pull request #8595 (2019-04-16 23:36:14)

[ENHANCEMENT] Project#config should use EMBER_ENV as default environment when none is passed in
When no `env` is passed to `project#config`, it uses `undefined` which some devs have [found confusing](https://github.com/ember-cli/ember-cli/issues/7779).

I'm proposing using the environment from `EMBER_ENV` as the default if no `env` is supplied.

- [x] Tested in demo app
- [x] Wrote new test
- [x] Both fast and slow ember-cli tests pass

Closes https://github.com/ember-cli/ember-cli/issues/7779


Patch:
diff --git a/lib/models/project.js b/lib/models/project.js
index c1790f5f6..acc0c3d62 100644
--- a/lib/models/project.js
+++ b/lib/models/project.js
@@ -251,10 +251,11 @@ class Project {
     @return {Object}     Merged configuration object
    */
   config(env) {
-    let key = `${this.configPath()}|${env}`;
+    let _env = env === undefined ? process.env.EMBER_ENV : env;
+    let key = `${this.configPath()}|${_env}`;
     let config = this.configCache.get(key);
     if (config === undefined) {
-      config = this.configWithoutCache(env);
+      config = this.configWithoutCache(_env);
       this.configCache.set(key, config);
     }
     return _.cloneDeep(config);
@@ -264,7 +265,7 @@ class Project {
    * @private
    * @method configWithoutCache
    * @param  {String} env Environment name
-   * @return {Object}     Merged confiration object
+   * @return {Object}     Merged configuration object
    */
   configWithoutCache(env) {
     let configPath = this.configPath();


Test patch:
diff --git a/tests/unit/models/project-test.js b/tests/unit/models/project-test.js
index fd6a1f9d9..59f94db5b 100644
--- a/tests/unit/models/project-test.js
+++ b/tests/unit/models/project-test.js
@@ -87,6 +87,19 @@ describe('models/project.js', function() {
       expect(called).to.equal(true);
     });
 
+    it('config() with no args returns the config corresponding to the current EMBER_ENV', function() {
+      process.env.EMBER_ENV = 'testing';
+
+      project.config();
+      let configCacheKey;
+      for (const key of project.configCache.keys()) {
+        if (key.match('|')) {
+          configCacheKey = key;
+        }
+      }
+      expect(configCacheKey).to.match(/testing/i);
+    });
+
     describe('memoizes', function() {
       it('memoizes', function() {
         project.config('development');
Repository: ember-cli/ember-cli
Pull request #8664 (2019-05-27 17:29:36)

fix README ember min version


Patch:
diff --git a/blueprints/addon/files/README.md b/blueprints/addon/files/README.md
index a8290806d..3c8113c5b 100644
--- a/blueprints/addon/files/README.md
+++ b/blueprints/addon/files/README.md
@@ -7,7 +7,7 @@
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember.js v2.18 or above
+* Ember.js v3.4 or above
 * Ember CLI v2.13 or above
 * Node.js v8 or above
 
diff --git a/blueprints/module-unification-addon/files/README.md b/blueprints/module-unification-addon/files/README.md
index a8290806d..3c8113c5b 100644
--- a/blueprints/module-unification-addon/files/README.md
+++ b/blueprints/module-unification-addon/files/README.md
@@ -7,7 +7,7 @@
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember.js v2.18 or above
+* Ember.js v3.4 or above
 * Ember CLI v2.13 or above
 * Node.js v8 or above
 


Test patch:
diff --git a/tests/fixtures/addon/npm/README.md b/tests/fixtures/addon/npm/README.md
index 88353c29d..36bd8fd41 100644
--- a/tests/fixtures/addon/npm/README.md
+++ b/tests/fixtures/addon/npm/README.md
@@ -7,7 +7,7 @@ foo
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember.js v2.18 or above
+* Ember.js v3.4 or above
 * Ember CLI v2.13 or above
 * Node.js v8 or above
 
diff --git a/tests/fixtures/addon/yarn/README.md b/tests/fixtures/addon/yarn/README.md
index 88353c29d..36bd8fd41 100644
--- a/tests/fixtures/addon/yarn/README.md
+++ b/tests/fixtures/addon/yarn/README.md
@@ -7,7 +7,7 @@ foo
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember.js v2.18 or above
+* Ember.js v3.4 or above
 * Ember CLI v2.13 or above
 * Node.js v8 or above
 
diff --git a/tests/fixtures/module-unification-addon/npm/README.md b/tests/fixtures/module-unification-addon/npm/README.md
index 88353c29d..36bd8fd41 100644
--- a/tests/fixtures/module-unification-addon/npm/README.md
+++ b/tests/fixtures/module-unification-addon/npm/README.md
@@ -7,7 +7,7 @@ foo
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember.js v2.18 or above
+* Ember.js v3.4 or above
 * Ember CLI v2.13 or above
 * Node.js v8 or above
 
diff --git a/tests/fixtures/module-unification-addon/yarn/README.md b/tests/fixtures/module-unification-addon/yarn/README.md
index 88353c29d..36bd8fd41 100644
--- a/tests/fixtures/module-unification-addon/yarn/README.md
+++ b/tests/fixtures/module-unification-addon/yarn/README.md
@@ -7,7 +7,7 @@ foo
 Compatibility
 ------------------------------------------------------------------------------
 
-* Ember.js v2.18 or above
+* Ember.js v3.4 or above
 * Ember CLI v2.13 or above
 * Node.js v8 or above
Repository: ember-cli/ember-cli
Pull request #8900 (2019-10-23 20:43:02)

[BUGFIX] Include legacyDecorators eslint option by default
Ember supports the legacy decorators proposal, which originally
placed the decorators _before_ the export keywords. However, eslint
now throws errors if this is used, which is inline with the stage 2
spec that was rejected. This enables the legacyDecorators flag in
eslint to allow decorator placement anywhere.


Patch:
diff --git a/blueprints/app/files/.eslintrc.js b/blueprints/app/files/.eslintrc.js
index 27a975d71..50ca17569 100644
--- a/blueprints/app/files/.eslintrc.js
+++ b/blueprints/app/files/.eslintrc.js
@@ -3,7 +3,10 @@ module.exports = {
   parser: 'babel-eslint',
   parserOptions: {
     ecmaVersion: 2018,
-    sourceType: 'module'
+    sourceType: 'module',
+    ecmaFeatures: {
+      legacyDecorators: true
+    }
   },
   plugins: [
     'ember'


Test patch:
diff --git a/tests/fixtures/addon/.eslintrc.js b/tests/fixtures/addon/.eslintrc.js
index ae411fb34..ef47d4218 100644
--- a/tests/fixtures/addon/.eslintrc.js
+++ b/tests/fixtures/addon/.eslintrc.js
@@ -3,7 +3,10 @@ module.exports = {
   parser: 'babel-eslint',
   parserOptions: {
     ecmaVersion: 2018,
-    sourceType: 'module'
+    sourceType: 'module',
+    ecmaFeatures: {
+      legacyDecorators: true
+    }
   },
   plugins: [
     'ember'
diff --git a/tests/fixtures/app/.eslintrc.js b/tests/fixtures/app/.eslintrc.js
index df7e0b0e9..4e2716074 100644
--- a/tests/fixtures/app/.eslintrc.js
+++ b/tests/fixtures/app/.eslintrc.js
@@ -3,7 +3,10 @@ module.exports = {
   parser: 'babel-eslint',
   parserOptions: {
     ecmaVersion: 2018,
-    sourceType: 'module'
+    sourceType: 'module',
+    ecmaFeatures: {
+      legacyDecorators: true
+    }
   },
   plugins: [
     'ember'
Repository: ember-cli/ember-cli
Pull request #8901 (2019-10-23 23:06:03)

[BUGFIX beta] Merge with existing `EmberENV`
This allows Ember Inspector or other code in `index.html` before the vendor script tag to set values in `EmberENV`.

Turbo87: why is this labeled as "bugfix"? what bug does it fix?
chancancode: @turno87 currently, the output produced by Ember CLI prevents anything else from setting the EmberENV global (since it replaces it at the top of the vendor bundle). Since this is also the mechanism used by the inspector in 3.14+ to opt into more detailed debug information (“debug render tree” feature), this is currently preventing the inspector from working as designed.
chancancode: It’s not an inspector specific problem, just that this is probably the first time we noticed the problem since the inspector need to set this variable before ember loads. 

Patch:
diff --git a/lib/broccoli/vendor-prefix.js b/lib/broccoli/vendor-prefix.js
index ea8843249..e5f80de60 100644
--- a/lib/broccoli/vendor-prefix.js
+++ b/lib/broccoli/vendor-prefix.js
@@ -1,4 +1,11 @@
-window.EmberENV = {{EMBER_ENV}};
+window.EmberENV = (function(EmberENV, extra) {
+  for (var key in extra) {
+    EmberENV[key] = extra[key];
+  }
+
+  return EmberENV;
+})(window.EmberENV || {}, {{EMBER_ENV}});
+
 var runningTests = false;
 
 {{content-for 'vendor-prefix'}}


Test patch:
diff --git a/tests/acceptance/brocfile-smoke-test-slow.js b/tests/acceptance/brocfile-smoke-test-slow.js
index d4cb1f0cf..1b465ae06 100644
--- a/tests/acceptance/brocfile-smoke-test-slow.js
+++ b/tests/acceptance/brocfile-smoke-test-slow.js
@@ -53,7 +53,7 @@ describe('Acceptance: brocfile-smoke-test', function() {
 
       // Changes in ember-optional-features 0.7.0 cause all defined values in optional-features.json
       // to end up in EmberENV. jquery-integration is explicitly defined for non MU apps
-      let expected = 'window.EmberENV = {"asdflkmawejf":";jlnu3yr23","_JQUERY_INTEGRATION":false};';
+      let expected = '(window.EmberENV || {}, {"asdflkmawejf":";jlnu3yr23","_JQUERY_INTEGRATION":false});';
       expect(vendorContents).to.contain(expected, 'EmberENV should be in assets/vendor.js');
     })
   );
diff --git a/tests/unit/broccoli/default-packager/ember-cli-internal-test.js b/tests/unit/broccoli/default-packager/ember-cli-internal-test.js
index 5ca8c9bd0..8e5bfc549 100644
--- a/tests/unit/broccoli/default-packager/ember-cli-internal-test.js
+++ b/tests/unit/broccoli/default-packager/ember-cli-internal-test.js
@@ -156,7 +156,9 @@ describe('Default Packager: Ember CLI Internal', function() {
         'runningTests = true;\n\nif (window.Testem) {\n  window.Testem.hookIntoTestFramework();\n}'
       );
 
-      expect(vendorPrefixFileContent).to.contain('window.EmberENV = {};\nvar runningTests = false;');
+      expect(vendorPrefixFileContent).to.contain(
+        'window.EmberENV = (function(EmberENV, extra) {\n  for (var key in extra) {\n    EmberENV[key] = extra[key];\n  }\n\n  return EmberENV;\n})(window.EmberENV || {}, {});\n\nvar runningTests = false;'
+      );
       expect(vendorSuffixFileContent).to.equal('');
     })
   );
Repository: ember-cli/ember-cli
Pull request #8905 (2019-10-31 12:07:55)

Use production environment for `npm run build` / `yarn build` by default
Output from `npm run build` should be optimized by default to make sure people deploy optimized versions of Ember.

This ensures that even newcomers or people comparing frameworks will compare the optimized versions and not the development builds.

It also ensures that builds on serverless services will be optimized out of the box. See [now.sh](https://zeit.co/guides/deploying-ember-with-zeit-now/) for a most recent example.


Patch:
diff --git a/blueprints/app/files/package.json b/blueprints/app/files/package.json
index ce6042bc0..8e6dc00a2 100644
--- a/blueprints/app/files/package.json
+++ b/blueprints/app/files/package.json
@@ -11,7 +11,7 @@
     "test": "tests"
   },
   "scripts": {
-    "build": "ember build",
+    "build": "ember build --environment=production",
     "lint:hbs": "ember-template-lint .",
     "lint:js": "eslint .",
     "start": "ember serve",


Test patch:
diff --git a/tests/fixtures/addon/npm/package.json b/tests/fixtures/addon/npm/package.json
index 209727043..04353836c 100644
--- a/tests/fixtures/addon/npm/package.json
+++ b/tests/fixtures/addon/npm/package.json
@@ -13,7 +13,7 @@
     "test": "tests"
   },
   "scripts": {
-    "build": "ember build",
+    "build": "ember build --environment=production",
     "lint:hbs": "ember-template-lint .",
     "lint:js": "eslint .",
     "start": "ember serve",
diff --git a/tests/fixtures/addon/yarn/package.json b/tests/fixtures/addon/yarn/package.json
index 0ddf3cb8f..5e8e30559 100644
--- a/tests/fixtures/addon/yarn/package.json
+++ b/tests/fixtures/addon/yarn/package.json
@@ -13,7 +13,7 @@
     "test": "tests"
   },
   "scripts": {
-    "build": "ember build",
+    "build": "ember build --environment=production",
     "lint:hbs": "ember-template-lint .",
     "lint:js": "eslint .",
     "start": "ember serve",
diff --git a/tests/fixtures/app/npm/package.json b/tests/fixtures/app/npm/package.json
index 815b857f8..0d5498e57 100644
--- a/tests/fixtures/app/npm/package.json
+++ b/tests/fixtures/app/npm/package.json
@@ -11,7 +11,7 @@
     "test": "tests"
   },
   "scripts": {
-    "build": "ember build",
+    "build": "ember build --environment=production",
     "lint:hbs": "ember-template-lint .",
     "lint:js": "eslint .",
     "start": "ember serve",
diff --git a/tests/fixtures/app/yarn/package.json b/tests/fixtures/app/yarn/package.json
index 5b6571a74..b5ca0157c 100644
--- a/tests/fixtures/app/yarn/package.json
+++ b/tests/fixtures/app/yarn/package.json
@@ -11,7 +11,7 @@
     "test": "tests"
   },
   "scripts": {
-    "build": "ember build",
+    "build": "ember build --environment=production",
     "lint:hbs": "ember-template-lint .",
     "lint:js": "eslint .",
     "start": "ember serve",
Repository: ember-cli/ember-cli
Pull request #8910 (2019-11-06 10:08:59)

restrict app Travis to master and PRs
We did this to addons, where they are restricted to master, PRs, and npm tags. I think it makes sense to do the same with apps, minus the npm tags?


Patch:
diff --git a/blueprints/app/files/.travis.yml b/blueprints/app/files/.travis.yml
index 25dc4c779..f85d65a78 100644
--- a/blueprints/app/files/.travis.yml
+++ b/blueprints/app/files/.travis.yml
@@ -20,6 +20,10 @@ env:
   global:
     # See https://git.io/vdao3 for details.
     - JOBS=1
+
+branches:
+  only:
+    - master
 <% if (yarn) { %>
 before_install:
   - curl -o- -L https://yarnpkg.com/install.sh | bash


Test patch:
diff --git a/tests/fixtures/app/npm/.travis.yml b/tests/fixtures/app/npm/.travis.yml
index 1fe0f1fca..abc5c8860 100644
--- a/tests/fixtures/app/npm/.travis.yml
+++ b/tests/fixtures/app/npm/.travis.yml
@@ -18,6 +18,10 @@ env:
     # See https://git.io/vdao3 for details.
     - JOBS=1
 
+branches:
+  only:
+    - master
+
 script:
   - npm run lint:hbs
   - npm run lint:js
diff --git a/tests/fixtures/app/yarn/.travis.yml b/tests/fixtures/app/yarn/.travis.yml
index e8232192c..eaab6fbba 100644
--- a/tests/fixtures/app/yarn/.travis.yml
+++ b/tests/fixtures/app/yarn/.travis.yml
@@ -17,6 +17,10 @@ env:
     # See https://git.io/vdao3 for details.
     - JOBS=1
 
+branches:
+  only:
+    - master
+
 before_install:
   - curl -o- -L https://yarnpkg.com/install.sh | bash
   - export PATH=$HOME/.yarn/bin:$PATH
Repository: ember-cli/ember-cli
Pull request #8917 (2019-11-09 19:20:59)

give npm a floating deps test too


Patch:
diff --git a/blueprints/addon/files/.travis.yml b/blueprints/addon/files/.travis.yml
index e4a1ab606..a124617a3 100644
--- a/blueprints/addon/files/.travis.yml
+++ b/blueprints/addon/files/.travis.yml
@@ -38,17 +38,18 @@ jobs:
     # runs linting and tests with current locked deps
 
     - stage: "Tests"
-      name: "Tests"<% if (yarn) { %>
-      install:
-        - yarn install --non-interactive<% } %>
+      name: "Tests"
       script:
         - <% if (yarn) { %>yarn lint:hbs<% } else { %>npm run lint:hbs<% } %>
         - <% if (yarn) { %>yarn lint:js<% } else { %>npm run lint:js<% } %>
-        - <% if (yarn) { %>yarn test<% } else { %>npm test<% } %><% if (yarn) { %>
+        - <% if (yarn) { %>yarn test<% } else { %>npm test<% } %>
 
     - name: "Floating Dependencies"
+      install:<% if (yarn) { %>
+        - yarn install --no-lockfile --non-interactive
       script:
-        - yarn test<% } %>
+        - yarn test<% } else { %>
+        - npm install --no-package-lock<% } %>
 
     # we recommend new addons test the current and previous LTS
     # as well as latest stable release (bonus points to beta/canary)
@@ -65,7 +66,7 @@ before_install:
   - export PATH=$HOME/.yarn/bin:$PATH
 
 install:
-  - yarn install --no-lockfile --non-interactive
+  - yarn install --non-interactive
 <% } %>
 script:
   - node_modules/.bin/ember try:one $EMBER_TRY_SCENARIO


Test patch:
diff --git a/tests/fixtures/addon/npm/.travis.yml b/tests/fixtures/addon/npm/.travis.yml
index a45eb4a56..a19c80a85 100644
--- a/tests/fixtures/addon/npm/.travis.yml
+++ b/tests/fixtures/addon/npm/.travis.yml
@@ -41,6 +41,10 @@ jobs:
         - npm run lint:js
         - npm test
 
+    - name: "Floating Dependencies"
+      install:
+        - npm install --no-package-lock
+
     # we recommend new addons test the current and previous LTS
     # as well as latest stable release (bonus points to beta/canary)
     - stage: "Additional Tests"
diff --git a/tests/fixtures/addon/yarn/.travis.yml b/tests/fixtures/addon/yarn/.travis.yml
index ab3e592e5..5cf2b5e9b 100644
--- a/tests/fixtures/addon/yarn/.travis.yml
+++ b/tests/fixtures/addon/yarn/.travis.yml
@@ -35,14 +35,14 @@ jobs:
 
     - stage: "Tests"
       name: "Tests"
-      install:
-        - yarn install --non-interactive
       script:
         - yarn lint:hbs
         - yarn lint:js
         - yarn test
 
     - name: "Floating Dependencies"
+      install:
+        - yarn install --no-lockfile --non-interactive
       script:
         - yarn test
 
@@ -61,7 +61,7 @@ before_install:
   - export PATH=$HOME/.yarn/bin:$PATH
 
 install:
-  - yarn install --no-lockfile --non-interactive
+  - yarn install --non-interactive
 
 script:
   - node_modules/.bin/ember try:one $EMBER_TRY_SCENARIO
Repository: ember-cli/ember-cli
Pull request #8929 (2019-11-15 19:31:09)

Drop Node 11 support in generated apps and addons

rwjblue: Eeck, sorry that I missed this for so long! Would you mind rebasing?

Patch:
diff --git a/blueprints/app/files/package.json b/blueprints/app/files/package.json
index 3cbda4146..21c0a53b0 100644
--- a/blueprints/app/files/package.json
+++ b/blueprints/app/files/package.json
@@ -48,7 +48,7 @@
     "qunit-dom": "^0.9.2"
   },
   "engines": {
-    "node": "8.* || >= 10.*"
+    "node": "8.* || 10.* || >= 12"
   },
   "ember": {
     "edition": "octane"


Test patch:
diff --git a/tests/fixtures/addon/npm/package.json b/tests/fixtures/addon/npm/package.json
index a0ba93de7..943d0544c 100644
--- a/tests/fixtures/addon/npm/package.json
+++ b/tests/fixtures/addon/npm/package.json
@@ -52,7 +52,7 @@
     "qunit-dom": "^0.9.2"
   },
   "engines": {
-    "node": "8.* || >= 10.*"
+    "node": "8.* || 10.* || >= 12"
   },
   "ember": {
     "edition": "octane"
diff --git a/tests/fixtures/addon/yarn/package.json b/tests/fixtures/addon/yarn/package.json
index 9b3db2125..07e7bd213 100644
--- a/tests/fixtures/addon/yarn/package.json
+++ b/tests/fixtures/addon/yarn/package.json
@@ -53,7 +53,7 @@
     "qunit-dom": "^0.9.2"
   },
   "engines": {
-    "node": "8.* || >= 10.*"
+    "node": "8.* || 10.* || >= 12"
   },
   "ember": {
     "edition": "octane"
diff --git a/tests/fixtures/app/npm/package.json b/tests/fixtures/app/npm/package.json
index 7cefc29a9..3edb2e1d1 100644
--- a/tests/fixtures/app/npm/package.json
+++ b/tests/fixtures/app/npm/package.json
@@ -47,7 +47,7 @@
     "qunit-dom": "^0.9.2"
   },
   "engines": {
-    "node": "8.* || >= 10.*"
+    "node": "8.* || 10.* || >= 12"
   },
   "ember": {
     "edition": "octane"
diff --git a/tests/fixtures/app/yarn/package.json b/tests/fixtures/app/yarn/package.json
index c7ef12dea..9bbe31735 100644
--- a/tests/fixtures/app/yarn/package.json
+++ b/tests/fixtures/app/yarn/package.json
@@ -48,7 +48,7 @@
     "qunit-dom": "^0.9.2"
   },
   "engines": {
-    "node": "8.* || >= 10.*"
+    "node": "8.* || 10.* || >= 12"
   },
   "ember": {
     "edition": "octane"
Repository: ember-cli/ember-cli
Pull request #8943 (2019-11-26 09:30:09)

Travis.yml: use fast_finish instead of undocumented fail_fast
`fail_fast` used in `.travis.yml` config for addons is undocumented and I'm not sure if it even works. Travis config online validator complains about it. The closest option to what we would like to achieve is [`fast_finish`](https://docs.travis-ci.com/user/customizing-the-build#fast-finishing).

Closes: https://github.com/ember-cli/ember-cli/issues/8912

tniezurawski: > Build execution time has reached the maximum allowed time for your plan (60 minutes).

🤔 It took AppVeyor more than 1 hour to test everything. So we have a timeout... Is it happening a lot? What should I do?
tniezurawski: @rwjblue Could you take a look on that?

Patch:
diff --git a/blueprints/addon/files/.travis.yml b/blueprints/addon/files/.travis.yml
index 94a99847f..f304fdc46 100644
--- a/blueprints/addon/files/.travis.yml
+++ b/blueprints/addon/files/.travis.yml
@@ -29,7 +29,7 @@ branches:
     - /^v\d+\.\d+\.\d+/
 
 jobs:
-  fail_fast: true
+  fast_finish: true
   allow_failures:
     - env: EMBER_TRY_SCENARIO=ember-canary
 


Test patch:
diff --git a/tests/fixtures/addon/npm/.travis.yml b/tests/fixtures/addon/npm/.travis.yml
index 9a5ca3dfe..591494cae 100644
--- a/tests/fixtures/addon/npm/.travis.yml
+++ b/tests/fixtures/addon/npm/.travis.yml
@@ -26,7 +26,7 @@ branches:
     - /^v\d+\.\d+\.\d+/
 
 jobs:
-  fail_fast: true
+  fast_finish: true
   allow_failures:
     - env: EMBER_TRY_SCENARIO=ember-canary
 
diff --git a/tests/fixtures/addon/yarn/.travis.yml b/tests/fixtures/addon/yarn/.travis.yml
index 244783ddb..0f5e4db21 100644
--- a/tests/fixtures/addon/yarn/.travis.yml
+++ b/tests/fixtures/addon/yarn/.travis.yml
@@ -25,7 +25,7 @@ branches:
     - /^v\d+\.\d+\.\d+/
 
 jobs:
-  fail_fast: true
+  fast_finish: true
   allow_failures:
     - env: EMBER_TRY_SCENARIO=ember-canary
Repository: ember-cli/ember-cli
Pull request #8944 (2019-11-26 10:03:01)

Travis.yml: Remove deprecated `sudo: false` option
According to this post: https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration#timeline---its-happening-fast. Using `sudo: false` is now deprecated.

Closes: https://github.com/ember-cli/ember-cli/issues/8940


Patch:
diff --git a/blueprints/addon/files/.travis.yml b/blueprints/addon/files/.travis.yml
index a124617a3..86efbeec2 100644
--- a/blueprints/addon/files/.travis.yml
+++ b/blueprints/addon/files/.travis.yml
@@ -5,7 +5,6 @@ node_js:
   # so that your addon works for all apps
   - "8"
 
-sudo: false
 dist: trusty
 
 addons:
diff --git a/blueprints/app/files/.travis.yml b/blueprints/app/files/.travis.yml
index f85d65a78..cd094a356 100644
--- a/blueprints/app/files/.travis.yml
+++ b/blueprints/app/files/.travis.yml
@@ -3,7 +3,6 @@ language: node_js
 node_js:
   - "8"
 
-sudo: false
 dist: trusty
 
 addons:


Test patch:
diff --git a/tests/fixtures/addon/npm/.travis.yml b/tests/fixtures/addon/npm/.travis.yml
index a19c80a85..b8434dec6 100644
--- a/tests/fixtures/addon/npm/.travis.yml
+++ b/tests/fixtures/addon/npm/.travis.yml
@@ -5,7 +5,6 @@ node_js:
   # so that your addon works for all apps
   - "8"
 
-sudo: false
 dist: trusty
 
 addons:
diff --git a/tests/fixtures/addon/yarn/.travis.yml b/tests/fixtures/addon/yarn/.travis.yml
index 5cf2b5e9b..9e56574ba 100644
--- a/tests/fixtures/addon/yarn/.travis.yml
+++ b/tests/fixtures/addon/yarn/.travis.yml
@@ -5,7 +5,6 @@ node_js:
   # so that your addon works for all apps
   - "8"
 
-sudo: false
 dist: trusty
 
 addons:
diff --git a/tests/fixtures/app/npm/.travis.yml b/tests/fixtures/app/npm/.travis.yml
index abc5c8860..8339c408e 100644
--- a/tests/fixtures/app/npm/.travis.yml
+++ b/tests/fixtures/app/npm/.travis.yml
@@ -3,7 +3,6 @@ language: node_js
 node_js:
   - "8"
 
-sudo: false
 dist: trusty
 
 addons:
diff --git a/tests/fixtures/app/yarn/.travis.yml b/tests/fixtures/app/yarn/.travis.yml
index eaab6fbba..b05f2ddba 100644
--- a/tests/fixtures/app/yarn/.travis.yml
+++ b/tests/fixtures/app/yarn/.travis.yml
@@ -3,7 +3,6 @@ language: node_js
 node_js:
   - "8"
 
-sudo: false
 dist: trusty
 
 addons: