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

[DOCS] Amendement de l'ADR sur les migrations #11625

Draft
wants to merge 6 commits into
base: dev
Choose a base branch
from

Conversation

yannbertrand
Copy link
Member

@yannbertrand yannbertrand commented Mar 11, 2025

🥓 Proposition

Depuis l'ADR pour formaliser nos travaux de migrations SQL tout semble fonctionner correctement.

On se rend compte que les down de nos migrations ne sont quasiment jamais joués, et surtout pas en production (ce qui est une bonne chose).

Avec la rigueur du code rétrocompatible, et pour ne plus avoir d'incohérence sur l'attendu, une partie des devs proposent d'assumer totalement de ne pas développer de "down".

Après synchro avec les captains, cette approche semble risquée.

On propose de préciser ici les différentes approches et la décision actuelle.

😋 Pour tester

N/A

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@aurelie-crouillebois
Copy link
Contributor

Les migrations doivent être rétrocompatibles ❤️

@yannbertrand
Copy link
Member Author

Rediscuté ce jour avec @BerengereC, je vais amender mon amendement car c'est trop simpliste.

A priori, en l'état actuel, les migrates down peuvent toujours dépanner en cas de grosse galère. Je préciserai le propos pour que tout le monde se fasse son idée 🙂

@yannbertrand yannbertrand force-pushed the tech-review-adr-migrations branch 2 times, most recently from 7bc5f98 to 4915ea0 Compare March 11, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cross-team Toutes les équipes de dev 👀 Tech Review Needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants