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

Change d'organisation pour les DECP #108

Merged
merged 1 commit into from May 19, 2020
Merged

Conversation

ColinMaudry
Copy link
Contributor

No description provided.

@ColinMaudry ColinMaudry requested a review from a team as a code owner May 19, 2020 14:16
@AntoineAugusti
Copy link
Contributor

Merci pour la PR.

Qu'on soit au clair @ColinMaudry, il n'y a pas de mécanisme de redirection entre l'ancien slug et le nouveau slug, pas comme sur GitHub. Ceci signifie que les anciens liens de schémas / de documentations vont envoyer vers une 404.

C'est ok ou ça pose un gros problème ?

@ColinMaudry
Copy link
Contributor Author

C'est OK, j'ai bien noté que je dois publier de nouveaux schemas.yml. Pour ce faire je vais publier des 1.4.1 et 1.5.1. Ça roule comme ça ?

@ColinMaudry
Copy link
Contributor Author

Ah non, ça va continuer à fail avec les 1.4.0 et 1.5.0. Bref, je vais faire en sorte que les tags de format-commande-publique renvoient des schemas.yml valides :)

@AntoineAugusti
Copy link
Contributor

Tu peux avoir des versions intermédiaires qui ne sont pas valides, ce n'est pas un problème. Elles ne seront pas publiées en revanche. En l'état, comme tes 2 tags sont très récents, je pense que tu peux les corriger et retaguer (gloups)

@ColinMaudry
Copy link
Contributor Author

OK je fais ça.

Par contre je bloque sur la manip git pour éditer 1.4.0 sur master... Je pensais avoir une branche 1.4, mais ça ne va pas marcher si les commits ne sont pas dans master, si ?

@AntoineAugusti
Copy link
Contributor

Il faut en effet que ce soit dans master et que tu aies un tag vers ce commit avec la bonne version. Tu devrais t'en sortir en faisant du rebase et force push

@ColinMaudry
Copy link
Contributor Author

Avec rebase + force push tu m'as perdu. C'est quoi la séquence ?

@AntoineAugusti
Copy link
Contributor

L'idée est de faire disparaitre des précédents commits et les remplacer par ce que tu juges valides. Tu peux faire tes commits de correction à la suite des commits sur master et les squash / fixup dans les précédents commits (pour faire comme si tu n'avais qu'un seul commit).

Tu peux lire :

Quand tu changes l'historique de Git, tu es obligé de force push git push -f comme tu écrases de l'historique.

@ColinMaudry
Copy link
Contributor Author

Merci, je vois globalement le principe, mais je ne l'ai jamais fait. À moi le pouvoir de la doc !

@ColinMaudry
Copy link
Contributor Author

1.4.0 et 1.5.0 sont maintenant OK. Il manque quelque chose pour merger ?

Copy link
Contributor

@AntoineAugusti AntoineAugusti left a comment

Choose a reason for hiding this comment

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

Non, je merge tout de suite

@AntoineAugusti AntoineAugusti merged commit 927f61e into master May 19, 2020
@AntoineAugusti AntoineAugusti deleted the ColinMaudry-NewOrga branch May 19, 2020 18:53
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

2 participants