Skip to content

Commit ab511e4

Browse files
committed
Update Ember 3.21
1 parent 2c37012 commit ab511e4

19 files changed

+5447
-979
lines changed

.editorconfig

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
root = true
66

7-
87
[*]
98
end_of_line = lf
109
charset = utf-8

.eslintrc.js

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,14 @@
1+
'use strict';
2+
13
module.exports = {
24
root: true,
5+
parser: 'babel-eslint',
36
parserOptions: {
47
ecmaVersion: 2018,
5-
sourceType: 'module'
8+
sourceType: 'module',
9+
ecmaFeatures: {
10+
legacyDecorators: true
11+
}
612
},
713
plugins: [
814
'ember'
@@ -14,8 +20,7 @@ module.exports = {
1420
env: {
1521
browser: true
1622
},
17-
rules: {
18-
},
23+
rules: {},
1924
overrides: [
2025
// node files
2126
{
@@ -44,9 +49,7 @@ module.exports = {
4449
node: true
4550
},
4651
plugins: ['node'],
47-
rules: Object.assign({}, require('eslint-plugin-node').configs.recommended.rules, {
48-
// add your custom rules and overrides for node files here
49-
})
52+
extends: ['plugin:node/recommended']
5053
}
5154
]
5255
};

.npmignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
/.env*
1313
/.eslintignore
1414
/.eslintrc.js
15+
/.git/
1516
/.gitignore
1617
/.template-lintrc.js
1718
/.travis.yml

.template-lintrc.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
'use strict';
22

33
module.exports = {
4-
extends: 'recommended',
5-
rules: {
6-
'attribute-indentation': false
7-
}
4+
extends: 'octane'
85
};

.travis.yml

Lines changed: 13 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,9 @@ language: node_js
33
node_js:
44
# we recommend testing addons with the same minimum supported node version as Ember CLI
55
# so that your addon works for all apps
6-
- "8"
6+
- "10"
77

8-
sudo: false
9-
dist: trusty
8+
dist: xenial
109

1110
addons:
1211
chrome: stable
@@ -26,41 +25,38 @@ branches:
2625
- /^v\d+\.\d+\.\d+/
2726

2827
jobs:
29-
fail_fast: true
28+
fast_finish: true
3029
allow_failures:
3130
- env: EMBER_TRY_SCENARIO=ember-canary
3231

3332
include:
3433
# runs linting and tests with current locked deps
35-
3634
- stage: "Tests"
3735
name: "Tests"
38-
install:
39-
- yarn install --non-interactive
4036
script:
41-
- yarn lint:hbs
42-
- yarn lint:js
43-
- yarn test
37+
- yarn lint
38+
- yarn test:ember
4439

45-
- name: "Floating Dependencies"
40+
- stage: "Additional Tests"
41+
name: "Floating Dependencies"
42+
install:
43+
- yarn install --no-lockfile --non-interactive
4644
script:
47-
- yarn test
45+
- yarn test:ember
4846

4947
# we recommend new addons test the current and previous LTS
5048
# as well as latest stable release (bonus points to beta/canary)
51-
- stage: "Additional Tests"
52-
env: EMBER_TRY_SCENARIO=ember-lts-3.11
49+
- env: EMBER_TRY_SCENARIO=ember-lts-3.12
50+
- env: EMBER_TRY_SCENARIO=ember-lts-3.16
5351
- env: EMBER_TRY_SCENARIO=ember-release
5452
- env: EMBER_TRY_SCENARIO=ember-beta
5553
- env: EMBER_TRY_SCENARIO=ember-canary
5654
- env: EMBER_TRY_SCENARIO=ember-default-with-jquery
55+
- env: EMBER_TRY_SCENARIO=ember-classic
5756

5857
before_install:
5958
- curl -o- -L https://yarnpkg.com/install.sh | bash
6059
- export PATH=$HOME/.yarn/bin:$PATH
6160

62-
install:
63-
- yarn install --no-lockfile --non-interactive
64-
6561
script:
6662
- node_modules/.bin/ember try:one $EMBER_TRY_SCENARIO

CONTRIBUTING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,4 +23,4 @@
2323
* `ember serve`
2424
* Visit the dummy application at [http://localhost:4200](http://localhost:4200).
2525

26-
For more information on using ember-cli, visit [https://ember-cli.com/](https://ember-cli.com/).
26+
For more information on using ember-cli, visit [https://ember-cli.com/](https://ember-cli.com/).

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ Compatibility
1515
* Ember Bootstrap v3 or above
1616
* Ember.js v3.11 or above
1717
* Ember CLI v2.13 or above
18-
* Node.js v8 or above
18+
* Node.js v10 or above
1919

2020
Please use `v1.x` if you need support for Ember <= 3.10.
2121

config/ember-try.js

Lines changed: 27 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,18 @@ module.exports = async function() {
77
useYarn: true,
88
scenarios: [
99
{
10-
name: 'ember-lts-3.11',
10+
name: 'ember-lts-3.4',
1111
npm: {
1212
devDependencies: {
13-
'ember-source': '~3.11.0'
13+
'ember-source': '~3.4.0'
14+
}
15+
}
16+
},
17+
{
18+
name: 'ember-lts-3.8',
19+
npm: {
20+
devDependencies: {
21+
'ember-source': '~3.8.0'
1422
}
1523
}
1624
},
@@ -38,7 +46,7 @@ module.exports = async function() {
3846
}
3947
}
4048
},
41-
// The default `.travis.yml` runs this scenario via `npm test`,
49+
// The default `.travis.yml` runs this scenario via `yarn test`,
4250
// not via `ember try`. It's still included here so that running
4351
// `ember try:each` manually or from a customized CI config will run it
4452
// along with all the other scenarios.
@@ -57,7 +65,22 @@ module.exports = async function() {
5765
},
5866
npm: {
5967
devDependencies: {
60-
'@ember/jquery': '^0.5.1'
68+
'@ember/jquery': '^1.1.0'
69+
}
70+
}
71+
},
72+
{
73+
name: 'ember-classic',
74+
env: {
75+
EMBER_OPTIONAL_FEATURES: JSON.stringify({
76+
'application-template-wrapper': true,
77+
'default-async-observers': false,
78+
'template-only-glimmer-components': false
79+
})
80+
},
81+
npm: {
82+
ember: {
83+
edition: 'classic'
6184
}
6285
}
6386
}

package.json

Lines changed: 32 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -15,52 +15,61 @@
1515
"test": "tests"
1616
},
1717
"scripts": {
18-
"build": "ember build",
18+
"build": "ember build --environment=production",
19+
"lint": "npm-run-all --aggregate-output --continue-on-error --parallel lint:*",
1920
"lint:hbs": "ember-template-lint .",
2021
"lint:js": "eslint .",
2122
"start": "ember server",
22-
"test": "ember test",
23-
"test:all": "ember try:each"
23+
"test": "npm-run-all lint:* test:*",
24+
"test:ember": "ember test",
25+
"test:ember-compatibility": "ember try:each"
2426
},
2527
"dependencies": {
2628
"chalk": "^2.0.1",
27-
"ember-cli-babel": "^7.7.3",
29+
"ember-cli-babel": "^7.22.1",
2830
"ember-cli-build-config-editor": "^0.5.0",
29-
"ember-cli-htmlbars": "^3.0.1",
31+
"ember-cli-htmlbars": "^5.3.1",
3032
"fs-extra": "^7.0.0",
3133
"rsvp": "^4.0.1"
3234
},
3335
"devDependencies": {
34-
"@ember/optional-features": "^0.7.0",
36+
"@ember/optional-features": "^2.0.0",
37+
"@glimmer/component": "^1.0.1",
38+
"@glimmer/tracking": "^1.0.1",
39+
"babel-eslint": "^10.1.0",
3540
"bootstrap": "^4.0.0",
3641
"broccoli-asset-rev": "^3.0.0",
42+
"ember-auto-import": "^1.6.0",
3743
"ember-bootstrap": "^3.0.0-rc.5",
38-
"ember-cli": "~3.12.0",
39-
"ember-cli-dependency-checker": "^3.1.0",
40-
"ember-cli-eslint": "^5.1.0",
41-
"ember-cli-htmlbars-inline-precompile": "^2.1.0",
42-
"ember-cli-inject-live-reload": "^2.0.1",
44+
"ember-cli": "~3.21.0",
45+
"ember-cli-dependency-checker": "^3.2.0",
46+
"ember-cli-inject-live-reload": "^2.0.2",
4347
"ember-cli-sri": "^2.1.1",
44-
"ember-cli-template-lint": "^1.0.0-beta.1",
45-
"ember-cli-uglify": "^2.1.0",
48+
"ember-cli-uglify": "^3.0.0",
4649
"ember-disable-prototype-extensions": "^1.1.3",
47-
"ember-export-application-global": "^2.0.0",
48-
"ember-load-initializers": "^2.0.0",
50+
"ember-export-application-global": "^2.0.1",
51+
"ember-load-initializers": "^2.1.1",
4952
"ember-maybe-import-regenerator": "^0.1.6",
5053
"ember-power-select": "^3.0.0",
51-
"ember-qunit": "^4.4.1",
52-
"ember-resolver": "^5.0.1",
53-
"ember-source": "~3.12.0",
54-
"ember-source-channel-url": "^1.1.0",
55-
"ember-try": "^1.0.0",
56-
"eslint-plugin-ember": "^6.2.0",
57-
"eslint-plugin-node": "^9.0.1",
54+
"ember-qunit": "^4.6.0",
55+
"ember-resolver": "^8.0.2",
56+
"ember-source": "~3.21.1",
57+
"ember-source-channel-url": "^3.0.0",
58+
"ember-template-lint": "^2.11.0",
59+
"ember-try": "^1.4.0",
60+
"eslint": "^7.8.0",
61+
"eslint-plugin-ember": "^8.13.0",
62+
"eslint-plugin-node": "^11.1.0",
5863
"loader.js": "^4.7.0",
59-
"qunit-dom": "^0.8.4"
64+
"npm-run-all": "^4.1.5",
65+
"qunit-dom": "^1.4.0"
6066
},
6167
"engines": {
6268
"node": "8.* || >= 10.*"
6369
},
70+
"ember": {
71+
"edition": "octane"
72+
},
6473
"ember-addon": {
6574
"configPath": "tests/dummy/config"
6675
}

testem.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
'use strict';
2+
13
module.exports = {
24
test_page: 'tests/index.html?hidepassed',
35
disable_watching: true,
@@ -7,6 +9,7 @@ module.exports = {
79
launch_in_dev: [
810
'Chrome'
911
],
12+
browser_start_timeout: 120,
1013
browser_args: {
1114
Chrome: {
1215
ci: [

tests/dummy/app/app.js

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,12 @@
11
import Application from '@ember/application';
2-
import Resolver from './resolver';
2+
import Resolver from 'ember-resolver';
33
import loadInitializers from 'ember-load-initializers';
4-
import config from './config/environment';
4+
import config from 'dummy/config/environment';
55

6-
const App = Application.extend({
7-
modulePrefix: config.modulePrefix,
8-
podModulePrefix: config.podModulePrefix,
9-
Resolver
10-
});
6+
export default class App extends Application {
7+
modulePrefix = config.modulePrefix;
8+
podModulePrefix = config.podModulePrefix;
9+
Resolver = Resolver;
10+
}
1111

1212
loadInitializers(App, config.modulePrefix);
13-
14-
export default App;

tests/dummy/app/resolver.js

Lines changed: 0 additions & 3 deletions
This file was deleted.

tests/dummy/app/router.js

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
11
import EmberRouter from '@ember/routing/router';
2-
import config from './config/environment';
2+
import config from 'dummy/config/environment';
33

4-
const Router = EmberRouter.extend({
5-
location: config.locationType,
6-
rootURL: config.rootURL
7-
});
4+
export default class Router extends EmberRouter {
5+
location = config.locationType;
6+
rootURL = config.rootURL;
7+
}
88

99
Router.map(function() {
1010
});
11-
12-
export default Router;

tests/dummy/app/templates/components/.gitkeep

Whitespace-only changes.
Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
{
2+
"schemaVersion": "1.0.0",
3+
"packages": [
4+
{
5+
"name": "ember-cli",
6+
"version": "3.21.0",
7+
"blueprints": [
8+
{
9+
"name": "addon",
10+
"outputRepo": "https://github.com/ember-cli/ember-addon-output",
11+
"codemodsSource": "ember-addon-codemods-manifest@1",
12+
"isBaseBlueprint": true,
13+
"options": [
14+
"--yarn",
15+
"--no-welcome"
16+
]
17+
}
18+
]
19+
}
20+
]
21+
}
Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,6 @@
11
{
2-
"jquery-integration": false
2+
"application-template-wrapper": false,
3+
"default-async-observers": true,
4+
"jquery-integration": false,
5+
"template-only-glimmer-components": true
36
}

tests/dummy/config/targets.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const browsers = [
66
'last 1 Safari versions'
77
];
88

9-
const isCI = !!process.env.CI;
9+
const isCI = Boolean(process.env.CI);
1010
const isProduction = process.env.EMBER_ENV === 'production';
1111

1212
if (isCI || isProduction) {

tests/test-helper.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import Application from '../app';
2-
import config from '../config/environment';
1+
import Application from 'dummy/app';
2+
import config from 'dummy/config/environment';
33
import { setApplication } from '@ember/test-helpers';
44
import { start } from 'ember-qunit';
55

0 commit comments

Comments
 (0)