-
Notifications
You must be signed in to change notification settings - Fork 13
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
Migrations automatiques de base de données #1046
Conversation
✅ Deploy Preview for stylo-docs canceled.
|
Intéressant 👍🏻 |
1696816
to
7084e6d
Compare
5e538a8
to
04458d8
Compare
working-directory: ${{ inputs.app-dir }} | ||
run: npm test | ||
env: | ||
DATABASE_URL: mongodb://127.0.0.1:${{ job.services.mongodb.ports[27017] }}/stylo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
je n'ai jamais vu cette syntaxe job.services.mongodb.ports[27017]
, entre crochets c'est la valeur par défaut ? Si c'est une reference ce n'est pas plutôt le premier élément de ports ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai pris exemple dans ce dépôt :
Après avoir lu la documentation Mapping Docker host and service container ports.
04458d8
to
ca94868
Compare
✅ Deploy Preview for stylo-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
9d5973b
to
a29052d
Compare
a29052d
to
94197c5
Compare
…1046) Co-authored-by: Thomas Parisot <[email protected]> Co-authored-by: Guillaume Grossetie <[email protected]>
-tests
à sa suite)graphql/app.js
DATABASE_URL
) au lieu de trois (MONGO_SERVER_DB
,MONGO_SERVER_PORT
,MONGO_SERVER
)fixes #122