Skip to content

Commit

Permalink
build packages before linting to ensure ui lib is there
Browse files Browse the repository at this point in the history
  • Loading branch information
sitek94 committed Nov 26, 2023
1 parent 60b8694 commit 28249fa
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
12 changes: 9 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,9 @@ jobs:
- name: 📥 Install Dependencies
run: pnpm install

- name: 📦 Build libs
run: pnpm build:ui

- name: 🔬 Lint
run: pnpm lint

Expand All @@ -83,6 +86,9 @@ jobs:
- name: 📥 Install Dependencies
run: pnpm install

- name: 📦 Build libs
run: pnpm build:ui

- name: 🔎 Type check
run: pnpm typecheck

Expand Down Expand Up @@ -137,7 +143,7 @@ jobs:
run: pnpm test:e2e

build:
name: 🏗️ Build
name: 🏗️ Build apps
needs: [changes]
runs-on: ubuntu-latest
steps:
Expand All @@ -158,8 +164,8 @@ jobs:
- name: 📥 Install Dependencies
run: pnpm install

- name: 🏗️ Build
run: pnpm build
- name: 🏗️ Build apps
run: pnpm build:apps

deploy-nestjs:
name: 🐯 Deploy NestJS App
Expand Down
2 changes: 2 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,11 @@
"name": "pnpm-monorepo",
"private": true,
"scripts": {
"build:apps": "pnpm -F './apps/**' build",
"build:docs": "pnpm -F docs build",
"build:nestjs": "pnpm -F nestjs build",
"build:remix": "pnpm -F remix build",
"build:ui": "pnpm -F ui build",
"build": "pnpm run -r build",
"deploy:nestjs": "fly deploy --config ./apps/nestjs/fly.toml --dockerfile ./apps/nestjs/Dockerfile",
"deploy:remix": "fly deploy --config ./apps/remix/fly.toml --dockerfile ./apps/remix/Dockerfile",
Expand Down

0 comments on commit 28249fa

Please sign in to comment.