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

Facilitate and document per-branch databases #16

Merged
merged 1 commit into from Jan 7, 2022
Merged

Conversation

florimondmanca
Copy link
Collaborator

## Description

Documentation d'un workflow suggéré pour gérer la BDD de dev quand on jongle entre des branches qui y apportent des modifications.

Motivation

J'ai l'habitude de créer une BDD par branche qui nécessite des modifications à la BDD. En effet 👍

  • Ca permet de tester des évolutions de la DB à partir d'une DB "en l'état", comme cela se produit en prod (plutôt qu'à partir d'une DB "propre")
  • Ca permet d'éviter de devoir effacer la DB de dev en jonglant entre les branches (les notres ou celles d'autres personnes, pour de la review par ex), et ainsi perdre les éventuelles données de test qu'on y a mises.

Comme ce n'est pas forcément intuitif et que pour ma part ce procédé me convient très bien, je me suis dis qu'il serait d'autant mieux de le documenter pour que tout le monde en profite. :-)

@florimondmanca florimondmanca added the docs Improvements or additions to documentation label Jan 4, 2022
@florimondmanca
Copy link
Collaborator Author

Je confirme que ça me manque à ce jour, et ça me paraît peu polémique comme changement, donc je merge. :-)

@florimondmanca florimondmanca merged commit 9ab7515 into master Jan 7, 2022
@florimondmanca florimondmanca deleted the fm/branchdb branch January 7, 2022 13:04
Copy link
Contributor

@magopian magopian left a comment

Choose a reason for hiding this comment

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

Review en retard, mais c'est pas grave, c'est très bien comme ça (juste une petite incompréhension sur l'utilisation de la var d'env dans le .env)

@@ -1 +1 @@
APP_DATABASE_URL="postgresql+asyncpg://user:pass@localhost:5432/catalogage"
APP_DATABASE_URL="postgresql+asyncpg://user:pass@localhost:5432/${DB:-catalogage}"
Copy link
Contributor

Choose a reason for hiding this comment

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

Je ne comprend pas ${DB:-catalogage} : est-ce que ça veut dire que si la variable d'environnement DB est absente, alors ça utilise catalogage? C'est ça le :-?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants