Skip to content

fix(ci): run check workflow for app and lib #22

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
skulidropek merged 1 commit into from
Feb 11, 2026
Merged

fix(ci): run check workflow for app and lib #22

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 13 additions & 9 deletions .github/workflows/check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,10 @@ jobs:
- uses: actions/checkout@v6
- name: Install dependencies
uses: ./.github/actions/setup
- run: pnpm check
- name: Types (lib)
run: pnpm --filter @effect-template/lib typecheck
- name: Typecheck (app)
run: pnpm --filter ./packages/app check
- name: Typecheck (lib)
run: pnpm --filter ./packages/lib typecheck

lint:
name: Lint
Expand All @@ -50,9 +51,10 @@ jobs:
# See: https://github.com/ton-ai-core/vibecode-linter/issues (pending issue)
- name: Install global linter dependencies
run: npm install -g typescript @biomejs/biome
- run: pnpm lint
- name: Lint (app)
run: pnpm --filter ./packages/app lint
- name: Lint (lib)
run: pnpm --filter @effect-template/lib lint
run: pnpm --filter ./packages/lib lint

test:
name: Test
Expand All @@ -65,9 +67,10 @@ jobs:
# vibecode-linter uses npx internally for dependency checks (lint:tests runs first)
- name: Install global linter dependencies
run: npm install -g typescript @biomejs/biome
- run: pnpm test
- name: Test (app)
run: pnpm --filter ./packages/app test
- name: Test (lib)
run: pnpm --filter @effect-template/lib test
run: pnpm --filter ./packages/lib test

lint-effect:
name: Lint Effect-TS
Expand All @@ -77,6 +80,7 @@ jobs:
- uses: actions/checkout@v6
- name: Install dependencies
uses: ./.github/actions/setup
- run: pnpm lint:effect
- name: Lint Effect-TS (app)
run: pnpm --filter ./packages/app lint:effect
- name: Lint Effect-TS (lib)
run: pnpm --filter @effect-template/lib lint:effect
run: pnpm --filter ./packages/lib lint:effect
10 changes: 5 additions & 5 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,19 +9,19 @@
],
"scripts": {
"build": "pnpm --filter ./packages/app build",
"check": "pnpm --filter ./packages/app check",
"check": "pnpm --filter ./packages/app check && pnpm --filter ./packages/lib typecheck",
"changeset": "changeset",
"changeset-publish": "node -e \"if (!process.env.NPM_TOKEN) { console.log('Skipping publish: NPM_TOKEN is not set'); process.exit(0); }\" && changeset publish",
"changeset-version": "changeset version",
"clone": "pnpm --filter ./packages/app build && node packages/app/dist/main.js clone",
"docker-git": "pnpm --filter ./packages/app build:docker-git && node packages/app/dist/src/docker-git/main.js",
"list": "pnpm --filter ./packages/app build && node packages/app/dist/main.js list",
"dev": "pnpm --filter ./packages/app dev",
"lint": "pnpm --filter ./packages/app lint",
"lint": "pnpm --filter ./packages/app lint && pnpm --filter ./packages/lib lint",
"lint:tests": "pnpm --filter ./packages/app lint:tests",
"lint:effect": "pnpm --filter ./packages/app lint:effect",
"test": "pnpm --filter ./packages/app test",
"typecheck": "pnpm --filter ./packages/app typecheck",
"lint:effect": "pnpm --filter ./packages/app lint:effect && pnpm --filter ./packages/lib lint:effect",
"test": "pnpm --filter ./packages/app test && pnpm --filter ./packages/lib test",
"typecheck": "pnpm --filter ./packages/app typecheck && pnpm --filter ./packages/lib typecheck",
"start": "pnpm --filter ./packages/app start"
},
"devDependencies": {
Expand Down