Skip to content

Commit 6af0f1d

Browse files
authored
Merge pull request #917 from ember-fastboot/use-pnpm
move to pnpm
2 parents 14238e0 + d4c30c2 commit 6af0f1d

File tree

18 files changed

+17994
-18265
lines changed

18 files changed

+17994
-18265
lines changed

.github/workflows/ci.yml

Lines changed: 52 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -22,83 +22,89 @@ jobs:
2222
test-all-packages:
2323
name: Ember Tests
2424
runs-on: ${{ matrix.os }}
25+
timeout-minutes: 15
2526

2627
strategy:
28+
fail-fast: false
2729
matrix:
2830
node-version: [16.x, 14.x]
2931
os: [ubuntu-latest, windows-latest]
3032

3133
steps:
3234
- name: Checkout Code
33-
uses: actions/checkout@v2
35+
uses: actions/checkout@v3
36+
- uses: pnpm/action-setup@v2
37+
with:
38+
version: 7
3439
- name: Use Node.js ${{ matrix.node-version }}
35-
uses: actions/setup-node@v2
40+
uses: actions/setup-node@v3
3641
with:
3742
node-version: ${{ matrix.node-version }}
38-
cache: yarn
39-
- name: Yarn Install
40-
run: yarn install --ignore-engines --frozen-lockfile
41-
- name: Run ember-cli-fastboot Tests
42-
run: yarn workspace ember-cli-fastboot test:ember
43+
cache: pnpm
44+
- run: pnpm install --frozen-lockfile
45+
- run: pnpm test:ember-cli-fastboot
4346
- name: Run fastboot Tests
4447
if: ${{ matrix.os != 'windows-latest' }}
45-
run: yarn workspace fastboot test
46-
- name: Run fastboot-express-middleware Tests
47-
run: yarn workspace fastboot-express-middleware test
48-
- name: Run fastboot-app-server Tests
49-
run: yarn workspace fastboot-app-server test:mocha
48+
run: pnpm test:fastboot
49+
- run: pnpm test:fastboot-express-middleware
50+
- run: pnpm test:fastboot-app-server
5051

5152
integration-tests:
5253
name: Integration Tests
5354
runs-on: ${{ matrix.os }}
55+
timeout-minutes: 15
5456

5557
strategy:
58+
fail-fast: false
5659
matrix:
5760
node-version: [16.x, 14.x]
5861
os: [ubuntu-latest, windows-latest]
5962

6063
steps:
6164
- name: Checkout Code
62-
uses: actions/checkout@v2
65+
uses: actions/checkout@v3
66+
- uses: pnpm/action-setup@v2
67+
with:
68+
version: 7
6369
- name: Use Node.js ${{ matrix.node-version }}
64-
uses: actions/setup-node@v2
70+
uses: actions/setup-node@v3
6571
with:
6672
node-version: ${{ matrix.node-version }}
67-
cache: yarn
68-
- name: Yarn Install
69-
run: yarn install --ignore-engines --frozen-lockfile
70-
- name: Integration Tests
71-
run: yarn workspace integration-tests test
73+
cache: pnpm
74+
- run: pnpm install --frozen-lockfile
75+
- run: pnpm test:integration
7276

7377
test-packages:
7478
name: Test Packages
7579
runs-on: ${{ matrix.os }}
80+
timeout-minutes: 15
7681

7782
strategy:
83+
fail-fast: false
7884
matrix:
7985
node-version: [16.x, 14.x]
8086
os: [ubuntu-latest, windows-latest]
8187

8288
steps:
8389
- name: Checkout Code
84-
uses: actions/checkout@v2
90+
uses: actions/checkout@v3
91+
- uses: pnpm/action-setup@v2
92+
with:
93+
version: 7
8594
- name: Use Node.js ${{ matrix.node-version }}
86-
uses: actions/setup-node@v2
95+
uses: actions/setup-node@v3
8796
with:
8897
node-version: ${{ matrix.node-version }}
89-
cache: yarn
90-
- name: Yarn Install
91-
run: yarn install --ignore-engines --frozen-lockfile
92-
- name: Basic App
93-
run: yarn workspace basic-app test:mocha
94-
- name: Custom App
95-
run: yarn workspace custom-fastboot-app test:mocha
96-
- name: Run ember-cli-fastboot-testing-app Tests
97-
run: yarn workspace ember-cli-fastboot-testing-app test:ember
98+
cache: pnpm
99+
- run: pnpm install --frozen-lockfile
100+
- run: pnpm --filter basic-app test:mocha
101+
- run: pnpm --filter custom-fastboot-app test:mocha
102+
- run: pnpm --filter ember-cli-fastboot-testing-app test:ember
98103

99104
test-legacy-mocha:
100105
name: Legacy Mocha Tests - ${{ matrix.node-version }}
101106
runs-on: ubuntu-latest
107+
timeout-minutes: 15
102108

103109
strategy:
104110
fail-fast: false
@@ -107,17 +113,17 @@ jobs:
107113

108114
steps:
109115
- name: Checkout Code
110-
uses: actions/checkout@v2
116+
uses: actions/checkout@v3
117+
- uses: pnpm/action-setup@v2
118+
with:
119+
version: 7
111120
- name: Use Node.js ${{ matrix.node-version }}
112-
uses: volta-cli/action@v1
121+
uses: actions/setup-node@v3
113122
with:
114123
node-version: ${{ matrix.node-version }}
115-
- name: Yarn Install
116-
run: |
117-
yarn install --ignore-engines --frozen-lockfile
118-
- name: Run Mocha Tests
119-
run: |
120-
yarn workspace ember-cli-fastboot test:mocha
124+
cache: pnpm
125+
- run: pnpm install --frozen-lockfile
126+
- run: pnpm --filter ember-cli-fastboot test:mocha
121127

122128
try-scenarios:
123129
name: "ember-try for ember-cli-fastboot: ${{ matrix.ember-try-scenario }}"
@@ -144,11 +150,14 @@ jobs:
144150
- ember-release-no-deprecations
145151

146152
steps:
147-
- uses: actions/checkout@v2
148-
- uses: actions/setup-node@v2
153+
- uses: actions/checkout@v3
154+
- uses: pnpm/action-setup@v2
155+
with:
156+
version: 7
157+
- uses: actions/setup-node@v3
149158
with:
150159
node-version: 14
151-
cache: yarn
152-
- run: yarn install --ignore-engines --frozen-lockfile
160+
cache: pnpm
161+
- run: pnpm install --frozen-lockfile
153162
- name: test
154-
run: yarn workspace ember-cli-fastboot ember try:one ${{ matrix.ember-try-scenario }} --skip-cleanup
163+
run: pnpm --filter ember-cli-fastboot exec ember try:one ${{ matrix.ember-try-scenario }} --skip-cleanup

.gitignore

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,6 @@ node_modules/
1818
yarn-error.log
1919

2020
# ember-try
21-
/.node_modules.ember-try/
22-
/bower.json.ember-try
23-
/package.json.ember-try
21+
.node_modules.ember-try/
22+
bower.json.ember-try
23+
package.json.ember-try

package.json

Lines changed: 10 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -5,24 +5,20 @@
55
"type": "git",
66
"url": "[email protected]:ember-fastboot/ember-cli-fastboot.git"
77
},
8-
"workspaces": [
9-
"packages/*",
10-
"test-packages/*"
11-
],
128
"scripts": {
139
"test": "npm-run-all test:*",
14-
"test:ember-cli-fastboot": "yarn workspace ember-cli-fastboot test:ember",
15-
"test:fastboot": "yarn workspace fastboot test",
16-
"test:fastboot-express-middleware": "yarn workspace fastboot-express-middleware test",
17-
"test:fastboot-app-server": "yarn workspace fastboot-app-server test:mocha",
18-
"test:integration": "yarn workspace integration-tests test",
19-
"test:extra": "yarn workspace basic-app test:mocha && yarn workspace custom-fastboot-app test:mocha"
10+
"test:ember-cli-fastboot": "pnpm --filter ember-cli-fastboot test:ember",
11+
"test:fastboot": "pnpm --filter fastboot test",
12+
"test:fastboot-express-middleware": "pnpm --filter fastboot-express-middleware test",
13+
"test:fastboot-app-server": "pnpm --filter fastboot-app-server test:mocha",
14+
"test:integration": "pnpm --filter integration-tests test",
15+
"test:extra": "pnpm --filter basic-app test:mocha && pnpm --filter custom-fastboot-app test:mocha"
2016
},
2117
"devDependencies": {
18+
"@release-it-plugins/lerna-changelog": "^5.0.0",
19+
"@release-it-plugins/workspaces": "^3.2.0",
2220
"npm-run-all": "^4.1.5",
23-
"release-it": "^14.2.2",
24-
"release-it-lerna-changelog": "^3.1.0",
25-
"release-it-yarn-workspaces": "^2.0.0"
21+
"release-it": "^14.2.2"
2622
},
2723
"publishConfig": {
2824
"registry": "https://registry.npmjs.org"
@@ -33,7 +29,7 @@
3329
"infile": "CHANGELOG.md",
3430
"launchEditor": true
3531
},
36-
"release-it-yarn-workspaces": {
32+
"@release-it-plugins/workspaces": {
3733
"workspaces": [
3834
"packages/*"
3935
],

packages/ember-cli-fastboot/config/ember-try.js

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,7 @@ const { embroiderSafe, embroiderOptimized } = require('@embroider/test-setup');
55

66
module.exports = async function () {
77
return {
8-
useYarn: true,
9-
// don't use default `--no-lockfile` option, which has been causing these to fail
10-
buildManagerOptions(/* scenario */) {
11-
return ['--ignore-engines'];
12-
},
8+
usePnpm: true,
139
scenarios: [
1410
{
1511
name: 'ember-lts-3.16',

packages/ember-cli-fastboot/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,10 @@
5858
"body-parser": "^1.18.3",
5959
"broccoli-asset-rev": "^3.0.0",
6060
"broccoli-test-helper": "^1.5.0",
61-
"co": "4.6.0",
6261
"chai": "^4.3.4",
6362
"chai-fs": "^2.0.0",
6463
"chai-string": "^1.4.0",
64+
"co": "4.6.0",
6565
"ember-auto-import": "^2.2.1",
6666
"ember-cli": "~4.1.0",
6767
"ember-cli-addon-tests": "^0.11.1",
@@ -81,7 +81,7 @@
8181
"ember-source": "~3.26.1",
8282
"ember-source-channel-url": "^3.0.0",
8383
"ember-template-lint": "^3.2.0",
84-
"ember-try": "^1.4.0",
84+
"ember-try": "^2.0.0",
8585
"eslint": "^7.23.0",
8686
"eslint-config-prettier": "^8.1.0",
8787
"eslint-plugin-ember": "^10.3.0",

packages/fastboot-express-middleware/test/fixtures/empty-package-json/package.json

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

packages/fastboot/test/fixtures/app-with-dependencies/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"dependencies": {
3-
"foo": "0.0.0"
3+
"foo": "workspace:*"
44
},
55
"fastboot": {
66
"appName": "app-with-dependencies",
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
{
2-
"name": "foo"
2+
"name": "empty-package-json"
33
}

0 commit comments

Comments
 (0)