Skip to content

Commit

Permalink
Merge pull request #1361 from Turbo87/pnpm
Browse files Browse the repository at this point in the history
Use `pnpm` package manager
  • Loading branch information
Turbo87 authored Dec 8, 2021
2 parents 11c3d8d + 7bee37f commit 0eb2b22
Show file tree
Hide file tree
Showing 11 changed files with 16,198 additions and 15,425 deletions.
35 changes: 24 additions & 11 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,9 @@ on:
schedule:
- cron: '0 3 * * *' # daily, at 3am

env:
PNPM_VERSION: 6.19.0

jobs:
test:
strategy:
Expand All @@ -19,16 +22,21 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v2
- uses: actions/setup-node@v2
- uses: actions/checkout@v2.4.0
- uses: pnpm/action-setup@v2.0.1
with:
version: ${{ env.PNPM_VERSION }}

- uses: actions/setup-node@v2.4.1
with:
node-version: ${{ matrix.node }}
cache: pnpm

- run: yarn install --no-lockfile
- run: yarn lint
- run: yarn build
- run: yarn test:coverage --runInBand
- run: yarn test:ember
- run: pnpm install
- run: pnpm lint
- run: pnpm build
- run: pnpm test:coverage -- --runInBand
- run: pnpm test:ember

compatibility:
name: "Tests (Ember.js, ember-try: ${{ matrix.ember-try-scenario }}, Embroider: ${{ matrix.embroider }})"
Expand All @@ -43,12 +51,17 @@ jobs:
- ember-lts-3.28

steps:
- uses: actions/checkout@v2
- uses: actions/setup-node@v2
- uses: actions/checkout@v2.4.0
- uses: pnpm/action-setup@v2.0.1
with:
version: ${{ env.PNPM_VERSION }}

- uses: actions/setup-node@v2.4.1
with:
node-version: 12
cache: pnpm

- run: yarn install --frozen-lockfile
- run: node_modules/.bin/ember try:one --config-path tests/dummy/config/ember-try.js ${{ matrix.ember-try-scenario}}
- run: pnpm install
- run: pnpm ember try:one ${{ matrix.ember-try-scenario}} -- --config-path tests/dummy/config/ember-try.js
env:
USE_EMBROIDER: "${{ matrix.embroider }}"
16 changes: 12 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,20 +5,28 @@ on:
tags:
- 'v*'

env:
PNPM_VERSION: 6.19.0

jobs:
release:
name: release
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v2
- uses: actions/setup-node@v2
- uses: actions/checkout@v2.4.0
- uses: pnpm/action-setup@v2.0.1
with:
version: ${{ env.PNPM_VERSION }}

- uses: actions/setup-node@v2.4.1
with:
node-version: 14
registry-url: 'https://registry.npmjs.org'
cache: pnpm

- run: yarn install
- run: yarn build
- run: pnpm install
- run: pnpm build

- name: auto-dist-tag
run: npx auto-dist-tag@1 --write
Expand Down
1 change: 1 addition & 0 deletions .npmignore
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
bower.json
documentation.yml
ember-cli-build.js
pnpm-lock.yaml
RELEASE.md
rollup.config.js
testem.js
Expand Down
22 changes: 22 additions & 0 deletions .pnpm-debug.log
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
{
"0 debug pnpm:scope": {
"selected": 1
},
"1 error pnpm": {
"code": "ELIFECYCLE",
"errno": "ENOENT",
"syscall": "spawn",
"file": "sh",
"pkgid": "qunit-dom@2.0.0",
"stage": "test:coverage",
"script": "jest --coverage",
"pkgname": "qunit-dom",
"err": {
"name": "pnpm",
"message": "qunit-dom@2.0.0 test:coverage: `jest --coverage`\nspawn ENOENT",
"code": "ELIFECYCLE",
"stack": "pnpm: qunit-dom@2.0.0 test:coverage: `jest --coverage`\nspawn ENOENT\n at ChildProcess.<anonymous> (/Users/tbieniek/.volta/tools/image/node/14.18.1/pnpm-global/5/node_modules/.pnpm/pnpm@6.19.1/node_modules/pnpm/dist/pnpm.cjs:92977:22)\n at ChildProcess.emit (events.js:400:28)\n at maybeClose (internal/child_process.js:1058:16)\n at Process.ChildProcess._handle.onexit (internal/child_process.js:293:5)"
}
},
"2 warn pnpm:global": " Local package.json exists, but node_modules missing, did you mean to install?"
}
6 changes: 3 additions & 3 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
## How to run tests

```bash
yarn test
pnpm test
```

## How to generate the documentation
Expand All @@ -12,5 +12,5 @@ The contents of [API.md](API.md) are generated from comments in the source code.
To update API.md, run the following command:

```bash
yarn docs
```
pnpm docs
```
4 changes: 2 additions & 2 deletions RELEASE.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,13 +39,13 @@ Once the prep work is completed, the actual release is straight forward:
* Next, ensure that you have installed your projects dependencies:

```
yarn install
pnpm install
```

* And last (but not least 😁) do your release:

```
yarn release
pnpm release
```

[release-it](https://github.com/release-it/release-it/) manages the actual
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
"scripts": {
"build": "rollup -c",
"changelog": "lerna-changelog",
"docs": "yarn build && documentation build dist/qunit-dom.js --config documentation.yml -f md -o API.md",
"docs": "npm run build && documentation build dist/qunit-dom.js --config documentation.yml -f md -o API.md",
"lint": "eslint . --cache",
"prepublish": "rollup -c",
"release": "release-it",
Expand Down
Loading

0 comments on commit 0eb2b22

Please sign in to comment.