ajout options avancees #15

Merged
paidge merged 1 commits from dev/cfg_avancee into master 12 months ago
paidge commented 12 months ago
Collaborator

Ajout de nouvelles options permettant de paramétrer TarteAuCitron

Ajout de nouvelles options permettant de paramétrer TarteAuCitron
paidge added 1 commit 12 months ago
paidge merged commit 1f0450e9b1 into master 12 months ago
paidge deleted branch dev/cfg_avancee 12 months ago

allo 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

allo allo @paidge tu n'as pas répondu à ma première question : https://git.spip.net/spip-contrib-extensions/tarteaucitron/pulls/13#issuecomment-26912 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
Poster
Collaborator

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.

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.
Poster
Collaborator

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.

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.
The pull request has been merged as 1f0450e9b1.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.