From 4e85ed78cd2e941107fdf0e8e118e7bee550109d Mon Sep 17 00:00:00 2001 From: Lukas Taegert-Atkinson Date: Sat, 24 Sep 2022 07:03:04 +0200 Subject: [PATCH] chore(all): fix lint issues (#1270) --- .github/workflows/node-windows.yml | 4 +- .github/workflows/release.yml | 8 +- .github/workflows/validate.yml | 6 +- .npmrc | 1 + README.md | 10 +- package.json | 2 +- packages/alias/package.json | 2 +- packages/auto-install/package.json | 2 +- packages/babel/package.json | 2 +- packages/babel/test/as-input-plugin.js | 9 +- packages/babel/test/as-output-plugin.js | 9 +- .../test/fixtures/proposal-decorators/main.js | 2 + packages/buble/package.json | 2 +- packages/commonjs/package.json | 2 +- packages/commonjs/test/test.js | 7 +- packages/data-uri/package.json | 4 +- packages/dsv/package.json | 2 +- packages/dynamic-import-vars/package.json | 4 +- packages/eslint/package.json | 2 +- packages/graphql/package.json | 2 +- packages/html/package.json | 2 +- packages/image/package.json | 2 +- packages/inject/package.json | 2 +- packages/json/package.json | 2 +- packages/legacy/package.json | 2 +- packages/multi-entry/package.json | 2 +- packages/node-resolve/package.json | 6 +- packages/pluginutils/package.json | 4 +- packages/pluginutils/src/dataToEsm.ts | 1 + packages/replace/package.json | 2 +- packages/replace/test/sourcemaps.js | 7 +- packages/run/package.json | 2 +- packages/strip/package.json | 2 +- packages/sucrase/package.json | 2 +- packages/typescript/package.json | 2 +- packages/url/package.json | 2 +- packages/virtual/package.json | 2 +- packages/wasm/package.json | 2 +- packages/yaml/package.json | 2 +- pnpm-lock.yaml | 418 +++++++++--------- 40 files changed, 286 insertions(+), 262 deletions(-) diff --git a/.github/workflows/node-windows.yml b/.github/workflows/node-windows.yml index 76014f4bb..b81b022bd 100644 --- a/.github/workflows/node-windows.yml +++ b/.github/workflows/node-windows.yml @@ -36,10 +36,10 @@ jobs: node-version: ${{ matrix.node }} - name: install pnpm - run: npm install pnpm@6 -g + run: npm install pnpm@7 -g - name: pnpm install run: pnpm install --ignore-scripts - name: run tests - run: pnpm ci:test --filter "...[origin/master]" + run: pnpm --filter "...[origin/master]" ci:test diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 1f9d54ec4..b71059471 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -43,7 +43,7 @@ jobs: - name: Install pnpm run: | - npm install pnpm@6 -g; + npm install pnpm@7 -g; echo node `pnpm -v`; - name: Set Git Config @@ -57,16 +57,16 @@ jobs: run: pnpm install - name: Build Packages - run: pnpm build --recursive + run: pnpm --recursive build - name: Lint Monorepo run: pnpm lint - name: Run Tests - run: pnpm test --filter [HEAD^] + run: pnpm --filter [HEAD^] test - name: Release and Publish Packages - run: pnpm release --filter [HEAD^] + run: pnpm --filter [HEAD^] release env: NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}} NPM_TOKEN: ${{ secrets.NPM_TOKEN }} diff --git a/.github/workflows/validate.yml b/.github/workflows/validate.yml index 779baaba8..0ede63214 100644 --- a/.github/workflows/validate.yml +++ b/.github/workflows/validate.yml @@ -34,7 +34,7 @@ jobs: run: git branch -f master origin/master - name: Install pnpm - run: npm install pnpm@6 -g + run: npm install pnpm@7 -g - name: Sanity Check run: | @@ -53,10 +53,10 @@ jobs: # run: pnpm security - name: Build Packages - run: pnpm build --recursive + run: pnpm --recursive build - name: Lint Monorepo run: pnpm lint:js - name: Run Tests - run: pnpm ci:coverage --filter "...[origin/master]" + run: pnpm --filter "...[origin/master]" ci:coverage diff --git a/.npmrc b/.npmrc index b500e9003..7328cfb89 100644 --- a/.npmrc +++ b/.npmrc @@ -4,3 +4,4 @@ enable-pre-post-scripts = true link-workspace-packages = false shamefully-hoist = true shared-workspace-shrinkwrap = true +strict-peer-dependencies = false diff --git a/README.md b/README.md index f08a02496..bcde44456 100644 --- a/README.md +++ b/README.md @@ -73,7 +73,7 @@ All plugin packages are kept in the `/packages` directory. #### Adding dependencies: ```console -$ pnpm add --filter ./packages/ +$ pnpm --filter ./packages/ add ``` Where `` is the name of the NPM package you wish to add for a plugin package, and `` is the proper name of the plugin. e.g. `@rollup/plugin-beep`. @@ -81,7 +81,7 @@ Where `` is the name of the NPM package you wish to add for a plugin pa #### Publishing: ```console -$ pnpm publish -- [flags] +$ pnpm publish [flags] ``` Where `` is the portion of the plugin package name following `@rollup/plugin-`. (e.g. `beep`) @@ -118,7 +118,7 @@ $ pnpm test To run tests on a specific package: ```console -$ pnpm test --filter ./packages/ +$ pnpm --filter ./packages/ test ``` Linting: @@ -132,10 +132,10 @@ $ pnpm lint To lint a specific package: ```console -$ pnpm lint --filter ./packages/ +$ pnpm --filter ./packages/ lint ``` -_Note: Scripts in the repository will run the root `test` and `lint` script on those packages which have changes. This is also how the CI pipelines function. To run either on a package outside of that pipeline, use `pnpm