définir les traitements avant de les appliquer #4855

Closed
JLuc wants to merge 5 commits from <deleted>:inclure_traitements into master
JLuc commented 1 year ago

Inclure les fichiers de fonctions yc inc/texte avant d'appliquer les traitements - Pour spip-contrib-extensions/crayons#10

Inclure les fichiers de fonctions yc inc/texte avant d'appliquer les traitements - Pour https://git.spip.net/spip-contrib-extensions/crayons/issues/10
JLuc added 5 commits 1 year ago
ecbcbfb931 master sync (#6)
JLuc commented 1 year ago
Poster

ce sera plus simple à refaire plutôt que de nettoyer.

ce sera plus simple à refaire plutôt que de nettoyer.
JLuc closed this pull request 1 year ago
Owner

ton master https://git.spip.net/JLuc/spip/commits/branch/master est tout pourri : tu as des commit de merge dessus, tu ne devrais jamais avoir ça

Pour faire proprement des PR tu dois partir d'un master propre, qui ne contient que les commits du repo d'origine. Pour cela tu ne dois jamais commit dessus, et tu le pull avec l'option --rebase pour ne pas avoir de commit de merge dessus non plus.

Ça permet ensuite d'avoir des branches, que tu peux rebase sur le master quand il a été mis a jour, et dont on aura ici que les commits ajoutés (et sans commit de merge)

ton master https://git.spip.net/JLuc/spip/commits/branch/master est tout pourri : tu as des commit de merge dessus, tu ne devrais jamais avoir ça Pour faire proprement des PR tu dois partir d'un master propre, qui ne contient que les commits du repo d'origine. Pour cela tu ne dois jamais commit dessus, et tu le pull avec l'option `--rebase` pour ne pas avoir de commit de merge dessus non plus. Ça permet ensuite d'avoir des branches, que tu peux rebase sur le master quand il a été mis a jour, et dont on aura ici que les commits ajoutés (et sans commit de merge)
Collaborator

les lignes suivantes dans le .gitconfig global évite ces commits de merge.

[pull]
	ff = only
	rebase = true
les lignes suivantes dans le .gitconfig global évite ces commits de merge. ``` [pull] ff = only rebase = true ```
JLuc deleted branch inclure_traitements 1 year ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Loading…
There is no content yet.