Skip to content

Commit

Permalink
fix: don't use env.docker-compose in ci directly
Browse files Browse the repository at this point in the history
  • Loading branch information
tefkah committed Feb 18, 2025
1 parent c3b0357 commit 8b2f9e4
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
5 changes: 1 addition & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,6 @@ jobs:
restore-keys: |
${{ runner.os }}-pnpm-store-
- name: Setup env file
run: cat ${{ env.ENV_FILE }} >> $GITHUB_ENV

- name: Cache turbo
uses: actions/cache@v4
with:
Expand All @@ -60,7 +57,7 @@ jobs:
${{ runner.os }}-turbo-
- name: Start test dependencies
run: docker compose -f docker-compose.test.yml --profile test up -d
run: pnpm test:setup

- name: Install dependencies
run: pnpm install --frozen-lockfile --prefer-offline
Expand Down
7 changes: 1 addition & 6 deletions .github/workflows/e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,6 @@ jobs:
restore-keys: |
${{ runner.os }}-pnpm-store-
- name: Setup env file
id: setup-env-file
run: cat ${{ env.ENV_FILE }} >> $GITHUB_OUTPUT

- name: Configure AWS credentials
uses: aws-actions/configure-aws-credentials@v4
with:
Expand Down Expand Up @@ -91,9 +87,8 @@ jobs:
run: pnpm p:build

- name: Run migrations and seed
run: pnpm --filter core exec prisma migrate reset --preview-feature --force
run: pnpm --filter core db:test:reset
env:
DATABASE_URL: postgresql://${{ steps.setup-env-file.outputs.POSTGRES_USER }}:${{ steps.setup-env-file.outputs.POSTGRES_PASSWORD }}@localhost:${{ steps.setup-env-file.outputs.POSTGRES_PORT }}/${{ steps.setup-env-file.outputs.POSTGRES_DB }}
# 20241126: this prevents the arcadia seed from running, which contains a ton of pubs which potentially might slow down the tests
MINIMAL_SEED: true
SKIP_VALIDATION: true
Expand Down
3 changes: 2 additions & 1 deletion core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@
"db:migrate-deploy": "pnpm migrate-deploy",
"db:migrate-diff": "pnpm migrate-diff",
"db:migrate-docker": "pnpm migrate-docker",
"db:migrate-test": "pnpm migrate-test",
"db:test:migrate": "pnpm migrate-test",
"db:test:reset": "dotenv -e .env.test -- prisma migrate reset --preview-feature --force | pino-pretty",
"db:prisma": "pnpm prisma",
"db:studio": "pnpm prisma studio",
"db:generate-history-table": "pnpm exec tsx prisma/scripts/history-tables/generate-history-table.mts",
Expand Down

0 comments on commit 8b2f9e4

Please sign in to comment.