Skip to content
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.

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

Add a possibility to test prisma migrations locally #996

Merged
merged 5 commits into from May 13, 2024

Conversation

JuliaBrigitte
Copy link
Contributor

@JuliaBrigitte JuliaBrigitte commented May 10, 2024

What does this change?

Add the possibility to test prima migrations locally

Why?

The feedback cycle is otherwise too long

How has it been verified?

Tested locally

@JuliaBrigitte JuliaBrigitte requested review from a team as code owners May 10, 2024 12:27
@JuliaBrigitte JuliaBrigitte force-pushed the aajb/dev-prisma-migrate branch 2 times, most recently from 5a14628 to a2dec06 Compare May 10, 2024 12:32
@JuliaBrigitte JuliaBrigitte force-pushed the aajb/dev-prisma-migrate branch 4 times, most recently from 9c66990 to 7d3ed52 Compare May 10, 2024 14:19
JuliaBrigitte and others added 4 commits May 13, 2024 14:40
Co-authored-by: Akash <akash1810@users.noreply.github.com>
Co-authored-by: Akash <akash1810@users.noreply.github.com>
Co-authored-by: Ashleigh <ashcorr20@gmail.com>
Comment on lines +52 to +58
## Testing the prima migration container
To test the prisma migration container you can run the following script:

```sh
./packages/dev-environment/script/start-prisma-migrate-test
```
This will run the run-prisma-migrate.sh script.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonder if this documentation would be better situated next to the container in containers/prisma-migrate?

@JuliaBrigitte JuliaBrigitte merged commit 8ad7f56 into main May 13, 2024
3 checks passed
@JuliaBrigitte JuliaBrigitte deleted the aajb/dev-prisma-migrate branch May 13, 2024 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants