Add test and generate actions#3221
Open
brage-andreas wants to merge 3 commits into
Open
Conversation
Member
Author
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
brage-andreas
changed the title
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
836576b to
2d85120
Compare
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
2d85120 to
7ac6cff
Compare
brage-andreas
force-pushed
the
add-setup-database-action
branch
2 times, most recently
from
e7c20b4 to
800eecc
Compare
junlarsen
requested changes
May 14, 2026
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
800eecc to
8437525
Compare
brage-andreas
force-pushed
the
add-setup-database-action
branch
3 times, most recently
from
55f152c to
220ab3f
Compare
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
33b2b74 to
5f620b8
Compare
brage-andreas
changed the base branch from
graphite-base/3221
to
fix-deploy-dependance-on-lint
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
5f620b8 to
4be8038
Compare
brage-andreas
force-pushed
the
fix-deploy-dependance-on-lint
branch
from
130545c to
291d140
Compare
brage-andreas
force-pushed
the
add-setup-database-action
branch
2 times, most recently
from
07a0eb9 to
c75bb9b
Compare
brage-andreas
force-pushed
the
fix-deploy-dependance-on-lint
branch
from
291d140 to
774c542
Compare
brage-andreas
force-pushed
the
add-setup-database-action
branch
2 times, most recently
from
60d2402 to
5945c81
Compare
brage-andreas
force-pushed
the
fix-deploy-dependance-on-lint
branch
from
774c542 to
0c893a2
Compare
brage-andreas
changed the title
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
5945c81 to
97e7e0c
Compare
brage-andreas
force-pushed
the
fix-deploy-dependance-on-lint
branch
from
0c893a2 to
3a84317
Compare
junlarsen
requested changes
May 20, 2026
Comment on lines
+51
to
+54
| --health-cmd "pg_isready -U $POSTGRES_USER -d $POSTGRES_DB" \ | ||
| --health-interval 5s \ | ||
| --health-timeout 5s \ | ||
| --health-retries 10 \ |
Member
There was a problem hiding this comment.
Why are we not using the postgres server on the runner, but use a container? I think this is probably magnitudes slower
brage-andreas
force-pushed
the
fix-deploy-dependance-on-lint
branch
from
3a84317 to
8cbba37
Compare
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
97e7e0c to
18c1751
Compare
Remove the database-url input so callers cannot pass credentials that do not match the fixed Postgres password. Mirror CI pnpm and Node setup (pinned versions, frozen lockfile install) before Prisma commands. Co-authored-by: Brage <brage-andreas@users.noreply.github.com>
brage-andreas
force-pushed
the
add-setup-database-action
branch
from
18c1751 to
d58486e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.