Skip to content

Commit 37dd414

Browse files
committed
update with ember-cli-update to 3.28
1 parent 6821902 commit 37dd414

File tree

13 files changed

+131
-103
lines changed

13 files changed

+131
-103
lines changed

.eslintignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
# misc
1414
/coverage/
1515
!.*
16+
.*/
1617
.eslintcache
1718

1819
# ember-try

.eslintrc.js

Lines changed: 14 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -24,22 +24,15 @@ module.exports = {
2424
// node files
2525
{
2626
files: [
27-
'.eslintrc.js',
28-
'.stylelintrc.js',
29-
'.prettierrc.js',
30-
'.template-lintrc.js',
31-
'ember-cli-build.js',
32-
'index.js',
33-
'testem.js',
34-
'blueprints/*/index.js',
35-
'config/**/*.js',
36-
'tests/dummy/config/**/*.js',
37-
],
38-
excludedFiles: [
39-
'addon/**',
40-
'addon-test-support/**',
41-
'app/**',
42-
'tests/dummy/app/**',
27+
'./.eslintrc.js',
28+
'./.prettierrc.js',
29+
'./.template-lintrc.js',
30+
'./ember-cli-build.js',
31+
'./index.js',
32+
'./testem.js',
33+
'./blueprints/*/index.js',
34+
'./config/**/*.js',
35+
'./tests/dummy/config/**/*.js',
4336
],
4437
parserOptions: {
4538
sourceType: 'script',
@@ -51,5 +44,10 @@ module.exports = {
5144
plugins: ['node'],
5245
extends: ['plugin:node/recommended'],
5346
},
47+
{
48+
// Test files:
49+
files: ['tests/**/*-test.{js,ts}'],
50+
extends: ['plugin:qunit/recommended'],
51+
},
5452
],
5553
};

.github/workflows/ci.yml

Lines changed: 39 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -3,89 +3,76 @@ name: CI
33
on:
44
push:
55
branches:
6-
- master
76
- main
8-
- "v*"
9-
pull_request:
7+
- master
8+
pull_request: {}
109

11-
env:
12-
NODE_VERSION: 10
13-
PERCY_PARALLEL_NONCE: ${{ github.run_id }}-${{ github.run_number }}
14-
PERCY_PARALLEL_TOTAL: 1
15-
FORCE_COLOR: 1
10+
concurrency:
11+
group: ci-${{ github.head_ref || github.ref }}
12+
cancel-in-progress: true
1613

1714
jobs:
18-
lint:
19-
name: Linting
15+
test:
16+
name: "Tests"
2017
runs-on: ubuntu-latest
18+
env:
19+
PERCY_TOKEN: ee0a9d5c1122d6a21852edf19b5b309aaec18077fb3900c98995c90bc48ed240
20+
RUN_PERCY_TESTS: true
2121

2222
steps:
2323
- uses: actions/checkout@v2
24+
- uses: mansona/npm-lockfile-version@v1
2425
- uses: actions/setup-node@v2
2526
with:
26-
node-version: ${{ env.NODE_VERSION }}
27-
27+
node-version: 12.x
28+
cache: npm
2829
- run: npm i -g npm@7
2930
- run: npm ci
3031
- run: npm run lint
32+
- run: npx percy exec -- npm run test
3133

32-
floating-dependencies:
33-
name: Floating Dependencies
34-
runs-on: ubuntu-latest
35-
36-
steps:
37-
- uses: actions/checkout@v2
38-
- uses: actions/setup-node@v2
39-
with:
40-
node-version: ${{ env.NODE_VERSION }}
41-
42-
- run: npm i -g npm@7
43-
- run: npm install --no-package-lock
44-
- run: npm test
45-
46-
test:
47-
name: Tests
34+
floating:
35+
name: "Floating Dependencies"
4836
runs-on: ubuntu-latest
49-
env:
50-
PERCY_TOKEN: ee0a9d5c1122d6a21852edf19b5b309aaec18077fb3900c98995c90bc48ed240
51-
RUN_PERCY_TESTS: true
52-
5337

5438
steps:
5539
- uses: actions/checkout@v2
56-
- uses: mansona/npm-lockfile-version@v1
5740
- uses: actions/setup-node@v2
5841
with:
59-
node-version: ${{ env.NODE_VERSION }}
60-
42+
node-version: 12.x
43+
cache: npm
6144
- run: npm i -g npm@7
62-
- run: npm ci
63-
- run: npx percy exec -- npm run test
45+
- run: npm install --no-shrinkwrap
46+
- run: npm run test:ember
6447

6548
try-scenarios:
66-
name: "ember-try: ${{ matrix.ember-try-scenario }}"
49+
name: ${{ matrix.try-scenario }}
6750
runs-on: ubuntu-latest
68-
timeout-minutes: 10
69-
needs: test
51+
needs: 'test'
7052

7153
strategy:
54+
fail-fast: false
7255
matrix:
73-
ember-try-scenario:
74-
- 'ember-lts-3.16'
75-
- 'ember-lts-3.20'
76-
- 'ember-release'
77-
- 'ember-beta'
78-
- 'ember-canary'
79-
- 'ember-default-with-jquery'
80-
- 'ember-classic'
56+
try-scenario:
57+
- ember-lts-3.16
58+
- ember-lts-3.20
59+
- ember-lts-3.24
60+
- ember-lts-3.28
61+
- ember-release
62+
- ember-beta
63+
- ember-canary
64+
- ember-classic
65+
- ember-default-with-jquery
66+
- embroider-safe
67+
- embroider-optimized
8168

8269
steps:
8370
- uses: actions/checkout@v2
8471
- uses: actions/setup-node@v2
8572
with:
86-
node-version: ${{ env.NODE_VERSION }}
87-
73+
node-version: 12.x
74+
cache: npm
8875
- run: npm i -g npm@7
8976
- run: npm ci
90-
- name: test
91-
run: node_modules/.bin/ember try:one ${{ matrix.ember-try-scenario }} --skip-cleanup
77+
- name: Run Tests
78+
run: ./node_modules/.bin/ember try:one ${{ matrix.try-scenario }}

.npmignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@
1515
/.eslintrc.js
1616
/.git/
1717
/.gitignore
18+
/.prettierignore
19+
/.prettierrc.js
1820
/.template-lintrc.js
1921
/.travis.yml
2022
/.watchmanconfig
@@ -24,6 +26,7 @@
2426
/ember-cli-build.js
2527
/testem.js
2628
/tests/
29+
/yarn-error.log
2730
/yarn.lock
2831
.gitkeep
2932

.template-lintrc.js

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

33
module.exports = {
4-
extends: 'octane',
4+
extends: 'recommended',
55
};

CONTRIBUTING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
## Linting
1010

1111
* `npm run lint`
12-
* `npm run lint:js -- --fix`
12+
* `npm run lint:fix`
1313

1414
## Running tests
1515

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ This addon is intended to provide basic components for easier style coordination
1111
Compatibility
1212
------------------------------------------------------------------------------
1313

14-
* Ember.js v3.16 or above
15-
* Ember CLI v2.13 or above
16-
* Node.js v10 or above
14+
* Ember.js v3.24 or above
15+
* Ember CLI v3.24 or above
16+
* Node.js v12 or above
1717

1818

1919
Installation

config/ember-try.js

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict';
22

33
const getChannelURL = require('ember-source-channel-url');
4+
const { embroiderSafe, embroiderOptimized } = require('@embroider/test-setup');
45

56
module.exports = async function () {
67
return {
@@ -21,6 +22,22 @@ module.exports = async function () {
2122
},
2223
},
2324
},
25+
{
26+
name: 'ember-lts-3.24',
27+
npm: {
28+
devDependencies: {
29+
'ember-source': '~3.24.3',
30+
},
31+
},
32+
},
33+
{
34+
name: 'ember-lts-3.28',
35+
npm: {
36+
devDependencies: {
37+
'ember-source': '~3.28.0',
38+
},
39+
},
40+
},
2441
{
2542
name: 'ember-release',
2643
npm: {
@@ -68,11 +85,16 @@ module.exports = async function () {
6885
}),
6986
},
7087
npm: {
88+
devDependencies: {
89+
'ember-source': '~3.28.0',
90+
},
7191
ember: {
7292
edition: 'classic',
7393
},
7494
},
7595
},
96+
embroiderSafe(),
97+
embroiderOptimized(),
7698
],
7799
};
78100
};

ember-cli-build.js

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,5 +42,12 @@ module.exports = function (defaults) {
4242
behave. You most likely want to be modifying `./index.js` or app's build file
4343
*/
4444

45-
return app.toTree();
45+
const { maybeEmbroider } = require('@embroider/test-setup');
46+
return maybeEmbroider(app, {
47+
skipBabel: [
48+
{
49+
package: 'qunit',
50+
},
51+
],
52+
});
4653
};

package.json

Lines changed: 23 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
"scripts": {
2020
"build": "ember build --environment=production",
2121
"changelog": "auto-changelog --template changelog.template --unreleased-only --prepend --load-github-issue-data --github-cache-dir .changelog",
22-
"lint": "npm-run-all --aggregate-output --continue-on-error --parallel 'lint:!(fix)'",
22+
"lint": "npm-run-all --aggregate-output --continue-on-error --parallel \"lint:!(fix)\"",
2323
"lint:fix": "npm-run-all --aggregate-output --continue-on-error --parallel lint:*:fix",
2424
"lint:hbs": "ember-template-lint .",
2525
"lint:hbs:fix": "ember-template-lint . --fix",
@@ -33,12 +33,12 @@
3333
},
3434
"dependencies": {
3535
"@ember/render-modifiers": "^1.0.2",
36-
"@glimmer/component": "^1.0.3",
36+
"@glimmer/component": "^1.0.4",
3737
"broccoli-funnel": "^3.0.2",
3838
"broccoli-merge-trees": "^4.2.0",
3939
"broccoli-postcss": "^5.1.0",
40-
"ember-cli-babel": "^7.23.1",
41-
"ember-cli-htmlbars": "^5.3.2",
40+
"ember-cli-babel": "^7.26.10",
41+
"ember-cli-htmlbars": "^5.7.2",
4242
"ember-concurrency": "^1.2.1",
4343
"ember-svg-jar": "^2.3.3",
4444
"ember-test-waiters": "^2.1.3",
@@ -51,41 +51,43 @@
5151
},
5252
"devDependencies": {
5353
"@ember/optional-features": "^2.0.0",
54-
"@ember/test-helpers": "^2.2.0",
55-
"@glimmer/tracking": "^1.0.3",
54+
"@ember/test-helpers": "^2.6.0",
55+
"@embroider/test-setup": "^0.48.1",
56+
"@glimmer/tracking": "^1.0.4",
5657
"auto-changelog": "github:mansona/auto-changelog#epic",
5758
"babel-eslint": "^10.1.0",
5859
"broccoli-asset-rev": "^3.0.0",
5960
"ember-array-helper": "^5.1.0",
60-
"ember-cli": "~3.25.1",
61+
"ember-cli": "~3.28.5",
6162
"ember-cli-dependency-checker": "^3.2.0",
6263
"ember-cli-fastboot": "^2.2.2",
63-
"ember-cli-inject-live-reload": "^2.0.2",
64+
"ember-cli-inject-live-reload": "^2.1.0",
6465
"ember-cli-netlify": "^0.3.1",
6566
"ember-cli-sri": "^2.1.1",
6667
"ember-cli-stylelint": "^4.0.0",
67-
"ember-cli-terser": "^4.0.1",
68+
"ember-cli-terser": "^4.0.2",
6869
"ember-decorators-polyfill": "^1.1.5",
6970
"ember-disable-prototype-extensions": "^1.1.3",
7071
"ember-export-application-global": "^2.0.1",
7172
"ember-fetch": "^8.0.1",
7273
"ember-load-initializers": "^2.1.2",
7374
"ember-maybe-import-regenerator": "^0.1.6",
7475
"ember-native-dom-helpers": "^0.6.2",
75-
"ember-page-title": "^6.2.1",
76+
"ember-page-title": "^6.2.2",
7677
"ember-percy": "^1.6.0",
77-
"ember-qunit": "^5.1.2",
78-
"ember-resolver": "^8.0.2",
78+
"ember-qunit": "^5.1.5",
79+
"ember-resolver": "^8.0.3",
7980
"ember-sinon": "^2.2.0",
80-
"ember-source": "~3.25.4",
81+
"ember-source": "~3.28.8",
8182
"ember-source-channel-url": "^3.0.0",
82-
"ember-template-lint": "^2.18.1",
83+
"ember-template-lint": "^3.15.0",
8384
"ember-try": "^1.4.0",
84-
"eslint": "^7.20.0",
85-
"eslint-config-prettier": "^7.2.0",
86-
"eslint-plugin-ember": "^10.2.0",
85+
"eslint": "^7.32.0",
86+
"eslint-config-prettier": "^8.3.0",
87+
"eslint-plugin-ember": "^10.5.8",
8788
"eslint-plugin-node": "^11.1.0",
88-
"eslint-plugin-prettier": "^3.3.1",
89+
"eslint-plugin-prettier": "^3.4.1",
90+
"eslint-plugin-qunit": "^6.2.0",
8991
"field-guide": "^1.6.0",
9092
"field-guide-default-template": "^1.3.1",
9193
"lint-to-the-future": "^0.7.0",
@@ -95,15 +97,15 @@
9597
"np": "*",
9698
"npm-run-all": "^4.1.5",
9799
"prember": "^1.0.3",
98-
"prettier": "^2.2.1",
99-
"qunit": "^2.14.0",
100+
"prettier": "^2.5.1",
101+
"qunit": "^2.17.2",
100102
"qunit-dom": "^1.6.0",
101103
"stylelint-config-standard": "^20.0.0",
102104
"stylelint-declaration-strict-value": "^1.0.4",
103105
"stylelint-order": "^4.0.0"
104106
},
105107
"engines": {
106-
"node": "10.* || >= 12"
108+
"node": "12.* || 14.* || >= 16"
107109
},
108110
"ember": {
109111
"edition": "octane"

0 commit comments

Comments
 (0)