-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #618 from buschtoens/pnpm
pnpm
- Loading branch information
Showing
15 changed files
with
11,857 additions
and
13,875 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| # Because this library needs to be built, | ||
| # we can't easily point package.json files at the git repo for easy cross-repo testing. | ||
| # | ||
| # This workflow brings back that capability by placing the compiled assets on a "dist" branch | ||
| # (configurable via the "branch" option below) | ||
| name: Push dist | ||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| - master | ||
| jobs: | ||
| push-dist: | ||
| name: Push dist | ||
| runs-on: ubuntu-latest | ||
| timeout-minutes: 10 | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: pnpm/action-setup@v3 | ||
| with: | ||
| version: 8 | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 18 | ||
| cache: pnpm | ||
| - name: Install Dependencies | ||
| run: pnpm install --frozen-lockfile | ||
| - uses: kategengler/[email protected] | ||
| with: | ||
| branch: dist | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| working-directory: 'ember-lazy-mount' |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,11 @@ | ||
| # See https://help.github.com/ignore-files/ for more about ignoring files. | ||
| /.idea/ | ||
|
|
||
| # dependencies | ||
| /node_modules/ | ||
|
|
||
| # ember-try | ||
| /.node_modules.ember-try/ | ||
| /bower.json.ember-try | ||
| /npm-shrinkwrap.json.ember-try | ||
| /package.json.ember-try | ||
| /package-lock.json.ember-try | ||
| /yarn.lock.ember-try |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,9 @@ | ||
| # Docs: https://pnpm.io/npmrc | ||
| # https://github.com/emberjs/rfcs/pull/907 | ||
|
|
||
| # we don't want addons to be bad citizens of the ecosystem | ||
| auto-install-peers=false | ||
|
|
||
| # we want true isolation, | ||
| # if a dependency is not declared, we want an error | ||
| resolve-peers-from-workspace-root=false |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,25 +2,29 @@ | |
|
|
||
| ## Installation | ||
|
|
||
| - `git clone <repository-url>` | ||
| - `cd my-addon` | ||
| - `npm install` | ||
| - `git clone [email protected]:buschtoens/ember-lazy-mount.git` | ||
| - `cd ember-lazy-mount` | ||
| - `pnpm install` | ||
|
|
||
| ## Linting | ||
|
|
||
| - `npm run lint:hbs` | ||
| * `npm run lint:js` | ||
| - `npm run lint:js -- --fix` | ||
| Inside any of the packages you can run: | ||
|
|
||
| - `pnpm lint` | ||
| - `pnpm lint:fix` | ||
|
|
||
| ## Running tests | ||
|
|
||
| - `npm run test` – Runs the test suite on the current Ember version | ||
| - `npm run test:ember -- --server` – Runs the test suite in "watch mode" | ||
| - `npm run test:ember-compatibility` – Runs the test suite against multiple Ember versions | ||
| - `cd ember-lazy-mount && pnpm start` – Builds the addon in "watch mode" so changes picked up by test app. | ||
| - `cd test-app && ember test` – Runs the test suite on the current Ember version | ||
| - `cd test-app && ember test --server` – Runs the test suite in "watch mode" | ||
| - `cd test-app && ember try:each` – Runs the test suite against multiple Ember versions | ||
|
|
||
| ## Running the dummy application | ||
| During development, if you'd like test app to pick up changes in the addon, make sure to run both | ||
| `cd ember-lazy-mount && pnpm start` and `cd test-app && ember test --server` in different terminals. | ||
|
|
||
| - `npm run start` | ||
| - Visit the dummy application at [http://localhost:4200](http://localhost:4200). | ||
| ## Running the test application | ||
|
|
||
| For more information on using ember-cli, visit [https://cli.emberjs.com/release/](https://cli.emberjs.com/release/). | ||
| - `cd test-app && ember serve` | ||
| - Visit the test application at [http://localhost:4200](http://localhost:4200). | ||
| For more information on using ember-cli, visit [https://cli.emberjs.com/release/](https://cli.emberjs.com/release/). | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,21 +6,25 @@ | |
| "test-engine" | ||
| ], | ||
| "scripts": { | ||
| "build": "yarn workspace ember-lazy-mount run build", | ||
| "lint": "yarn workspaces run lint", | ||
| "lint:fix": "yarn workspaces run lint:fix", | ||
| "prepare": "yarn build", | ||
| "start": "concurrently 'yarn:start:*' --restart-after 5000 --prefix-colors cyan,white,yellow", | ||
| "start:addon": "yarn workspace ember-lazy-mount run start", | ||
| "start:test-app": "yarn workspace test-app run start", | ||
| "test": "yarn workspaces run test", | ||
| "test:ember": "yarn workspace test-app run test:ember" | ||
| "build": "concurrently 'pnpm:build:*'", | ||
| "build:js": "rollup --config", | ||
| "build:types": "glint --declaration", | ||
| "lint": "concurrently 'pnpm:lint:*(!fix)' --names 'lint:'", | ||
| "lint:fix": "concurrently 'pnpm:lint:*:fix' --names 'fix:'", | ||
| "lint:js": "eslint . --cache", | ||
| "lint:js:fix": "eslint . --fix", | ||
| "lint:types": "glint", | ||
| "prepack": "concurrently 'pnpm:build:*'", | ||
| "start": "concurrently 'pnpm:start:*'", | ||
| "start:js": "rollup --config --watch --no-watch.clearScreen", | ||
| "start:types": "glint --declaration --watch", | ||
| "test": "echo 'A v2 addon does not have tests, run tests in test-app'" | ||
| }, | ||
| "publishConfig": { | ||
| "registry": "https://registry.npmjs.org" | ||
| }, | ||
| "packageManager": "[email protected]", | ||
| "volta": { | ||
| "node": "18.20.4" | ||
| }, | ||
| "dependencies": {} | ||
| } | ||
| } | ||
Oops, something went wrong.