Skip to content

Commit

Permalink
ci: remove packageManager temporarily
Browse files Browse the repository at this point in the history
  • Loading branch information
JounQin committed Jul 6, 2022
1 parent a015a19 commit aebb59a
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 12 deletions.
7 changes: 1 addition & 6 deletions .github/workflows/pkg-size.yml
Expand Up @@ -14,11 +14,6 @@ jobs:
- name: Checkout
uses: actions/checkout@v3

- name: Setup pnpm
uses: pnpm/action-setup@v2
with:
version: latest

- name: Setup Node.js
uses: actions/setup-node@v3
with:
Expand All @@ -27,6 +22,6 @@ jobs:
- name: Package Size Report
uses: pkg-size/action@v1
with:
build-command: pnpm build
build-command: npx pnpm run build
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
1 change: 1 addition & 0 deletions .npmrc
@@ -1,3 +1,4 @@
enable-pre-post-scripts=true
; https://github.com/pkg-size/action/pull/45
; package-lock=false
public-hoist-pattern[]=@1stg/*
Expand Down
8 changes: 4 additions & 4 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

16 changes: 14 additions & 2 deletions tests/withJsExtension/test.js
Expand Up @@ -127,5 +127,17 @@ assertResolve(
path.relative(__dirname, require.resolve('typescript/lib/typescript.js')),
)

// resolves symlinks by default
assertResolve('dummy.js', '../../dummy.js/index.js')
try {
// resolves symlinks by default
assertResolve('dummy.js', '../../dummy.js/index.js')
} catch {
try {
// it could be `file:` protocol
assertResolve(
'dummy.js',
'../../node_modules/.pnpm/file+dummy.js/node_modules/dummy.js/index.js',
)
} catch {
assertResolve('dummy.js', '../../node_modules/dummy.js/index.js')
}
}

0 comments on commit aebb59a

Please sign in to comment.