ajout options avancees #15
Merged
paidge
merged 1 commits from dev/cfg_avancee
into master
2 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'dev/cfg_avancee'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Ajout de nouvelles options permettant de paramétrer TarteAuCitron
1f0450e9b1
into master 2 years agoallo allo @paidge tu n'as pas répondu à ma première question : #13
tu as refait pareil encore ici, ça fait deux fois que tu fusionnes ta propre PR immédiatement, avant même que des gens aient relu et commenté
ça sert à quoi de faire des PR du coup ? soit tu fusionnes directement en git (mais ça ne vaut normalement que pour les modifs évidentes bénignes ou quand on est la personne principale qui maintient), mais si on fait des PR c'est explicitement pour que les gens relisent et valident
Salut @rastapopoulos. Je n'ai pas été notifié de la première question. Je me disais qu'effectivement ce n'était sûrement pas une bonne idée de fusionner la branche directement. Tu as tout à fait raison. Je ne suis pas habitué à contribuer donc pardonne-moi pour cette initiative.
Les notifications sont arrivées dans le dossier SPAM de ma boîte aux lettres en fait. Encore désolé, je pensais bien faire avant d'être en vacances... Les prochaines fois, je ferai juste une demande de merge. C'est la première fois que j'utilise GIT sur un projet où je ne suis pas seul donc je dois apprendre les règles.
1f0450e9b1
.