You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].
Vous avez testé de build le projet avec vos modifications et ce build a réussi
Vous respectez les conventions de codage et de nommage du projet
Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
Cette pull request n'est pas un duplicata d'une autre
Cette pull request est prête à être revue (review) et fusionnée (merge)
Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
Les détails des changements ont été décrits ci-dessous
Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)
Changelogs proposés
Vérification que l'onglet "Discussions" est activé et affichage d'un message si ça n'est pas le cas
Correct en général mais l'indentation peut être amélioré !
J'ai accepté tes suggestions. Faudrait mettre à jour la config eslint pour que tout ça soit fait automatiquement.
OK parfait, je review ça. Pour eslint, c'est acceptable ce que t'avais fait.
Mais il faut retenir que faire une condition dans le return, il faut mettre des () pour être sûr d'une Indentation parfaite (c'est pour ça que je t'ai suggéré tout ça). Tu regardes la page Homeworks, il y a plein de problèmes d'indentation comme ça
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.
🚀 Nouvelle Pull Request
Proposez vos modifications pour améliorer Papillon
Informations importantes
Merci de vous référer à la documentation sur la contribution si vous avez des questions à propos des pull requests (https://gitbook.getpapillon.xyz/organisation/outils-internes/github)
Checklist d'avant pull request
Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].
TODO(aka des annotations pour du code manquant) dans vos modificationsChangelogs proposés
Vérification que l'onglet "Discussions" est activé et affichage d'un message si ça n'est pas le cas
Informations supplémentaires
Closes #613