Mis a Jour de l'ancienne Ligne Éditoriale vers la nouvelle #6342
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ajout de la syntax propose de @philippemilink Co-authored-by: Ph. SW. <philippe.milink@gmx.fr>
philippemilink
approved these changes
Jun 26, 2022
There was a problem hiding this comment.
QA OK ✔️
Cependant, quelques remarques sur la présentation de la PR :
- attention à l'orthographe : il manque une lettre et un accent dans le titre de la PR, il y a quelques majuscules en trop. Le titre pourrait plus se concentrer sur les changements apportés par la PR tout en étant plus court : Mise à jour du lien de la ligne éditoriale
- les instructions que tu as données pour faire la QA correspondent à la revue de code (qui est implicite quand on fait la QA). Il aurait fallu dire quelque chose comme : *Aller sur le formulaire pour créer un nouveau contenu, et constater que le lien vers la ligne éditoriale dans le bandeau vert mène vers la nouvelle version.
- crée-toi une branche par PR, qui ne contiendra que les commits dédiés à cette PR et qui sera à jour par rapport à la branche
dev. Ça permet de voir plus facilement quels sont les commits qui sont réellement en rapport avec ta PR.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contrôle qualité
./zds/settings/abstract_base/zds.py) et voyez la nouvelle lignes Éditoriale