Skip to content

Commit

Permalink
chore(.github/workflows): improve workflow with removing duplicated c…
Browse files Browse the repository at this point in the history
…onfigs (#2874)
  • Loading branch information
sukvvon authored Dec 19, 2024
1 parent 51a742d commit c9359fa
Show file tree
Hide file tree
Showing 9 changed files with 8 additions and 16 deletions.
1 change: 0 additions & 1 deletion .github/workflows/compressed-size-action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ jobs:
with:
node-version: 20
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- uses: preactjs/compressed-size-action@v2
with:
pattern: './dist/**/*.{js,mjs}'
2 changes: 1 addition & 1 deletion .github/workflows/cr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- run: corepack enable
- uses: pnpm/action-setup@v4
- uses: actions/setup-node@v4
with:
node-version: 20
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/ecosystem-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,7 @@ jobs:
with:
node-version: 22
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- run: pnpm install --frozen-lockfile
- run: pnpm install
- name: Get Short SHA
id: short_sha
run: |
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/lint-and-type.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,7 @@ jobs:
with:
node-version: 20
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- run: pnpm install --frozen-lockfile
- run: pnpm install
- run: pnpm test:format
- run: pnpm test:types
- run: pnpm test:lint
3 changes: 1 addition & 2 deletions .github/workflows/livecodes-preview.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,8 @@ jobs:
with:
node-version: 20
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- uses: live-codes/preview-in-livecodes@v1
with:
install-command: pnpm install --frozen-lockfile
install-command: pnpm install
build-command: pnpm build
base-url: 'https://{{LC::REF}}.preview-in-livecodes-demo.pages.dev'
3 changes: 1 addition & 2 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,7 @@ jobs:
node-version: 20
registry-url: 'https://registry.npmjs.org'
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- run: pnpm install --frozen-lockfile
- run: pnpm install
- run: pnpm test
- run: pnpm build
- run: npm publish
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/test-multiple-builds.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,7 @@ jobs:
with:
node-version: 20
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- run: pnpm install --frozen-lockfile
- run: pnpm install
- run: pnpm build
- name: Use React 17 for production test
if: ${{ matrix.env == 'production' }}
Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/test-multiple-versions.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,7 @@ jobs:
with:
node-version: 20
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- run: pnpm install --frozen-lockfile
- run: pnpm install
- run: pnpm build # we don't have any other workflows to test build
- run: pnpm test:spec

Expand Down
3 changes: 1 addition & 2 deletions .github/workflows/test-old-typescript.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,7 @@ jobs:
with:
node-version: 20
cache: 'pnpm'
cache-dependency-path: '**/pnpm-lock.yaml'
- run: pnpm install --frozen-lockfile
- run: pnpm install
- run: pnpm build
- name: Patch for all TS
run: |
Expand Down

0 comments on commit c9359fa

Please sign in to comment.