Added auto-deploy

[VForiel]

No description provided.

[Aeris1One]

Hé hé hé, je vois que mon format de commit se répands :)

[Aeris1One]

LGTM, je pense qu'il serait judicieux de tester d'abord l'Action uniquement sur bêta avant de l'activer aussi sur master.

[VForiel]

Yep mais tu m'as dis que les actions ne marchent qu'une fois placées sur la branche master non ?
Ou alors tu veux que je retire l'action master de la PR ?

[ascpial]

LGTM

[ascpial]

D'après Aeris one on peut dire que la branche beta est la branche principale et ça permet de tester l'action

[VForiel]

Pas bête ! Il y a un intérêt à avoir master comme branche principale ou bien on peut laisser beta en tant que branche principale de façon permanente ?

[ascpial]

La sécurité des branches ne s'applique pas à la branche principale ? Ou alors c'est au cas par cas ?

[Aeris1One]

À vrai dire je pensais commenter l'Action "deploy master" ici et merge sur master.

Mais maintenant qu'ascpial le dit on peut effectivement mettre la branche bêta en principale de façon permanente, ce qui permettrait de pouvoir mettre les Actions dessus, mais aussi la mettrais par défaut pour les nouvelles PR (je compte plus le nombre de fois où on m'as dit "change de branche t'as commit sur master" quand j'ouvrais des PR sur d'autres projets)

[ZRunner]

Pour la protection des branches, si on veut protéger main en gardant beta par défaut, la doc est ici : https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests. Il y a d’ailleurs un joli petit avertissement :

Protected branches are available in public repositories with GitHub Free and GitHub Free for organizations, and in public and private repositories with GitHub Pro, GitHub Team, GitHub Enterprise Cloud, and GitHub Enterprise Server

Mais je suis d’accord pour mettre bêta en défaut, c’est ce que j’ai fait pour Zbot d’ailleurs. Et en bonus, les stats de commits visible sur les profils des utilisateurs ne comptent que les branches par défaut donc ça motivera à commit :kappa:

[Aeris1One]

Ah ! C'est donc pour ça que j'ai 100 commits au total sur ma page de stats !
ouais nan peut-être pas la seule raison

[VForiel]

Mais je suis d’accord pour mettre bêta en défaut, c’est ce que j’ai fait pour Zbot d’ailleurs. Et en bonus, les stats de commits visible sur les profils des utilisateurs ne comptent que les branches par défaut donc ça motivera à commit :kappa:

@ZRunner ça compte les commits fais dans des PR ? Ou bien c'est les commits direct sur la branche par défaut ? (auquel cas c'est nul pck même sur beta on fait des PR)

[ZRunner]

ça compte les commits fais dans des PR ? Ou bien c'est les commits direct sur la branche par défaut ? (auquel cas c'est nul pck même sur beta on fait des PR)
Bah, quand une PR est validée, à moins qu’elle soit squash tous tes commits sont ajoutés à l’historique de la branche, donc ça les compte correctement.

[VForiel]

Bah, quand une PR est validée, à moins qu’elle soit squash tous tes commits sont ajoutés à l’historique de la branche, donc ça les compte correctement.

Mais du coup si on applique cette même logique, les PR faites sur beta ajoutent les commits à beta, puis le PR de beta à master ajoute ces mêmes commits à master du coup non ?

[ZRunner]

Mais du coup si on applique cette même logique, les PR faites sur beta ajoutent les commits à beta, puis le PR de beta à master ajoute ces mêmes commits à master du coup non ?

Je pensais qu'on squashait les PR sur master. Si c'est pas le cas alors oui, ça revient au même, au détail près que les contributions apparaîtront plus tard sur les profils (au moment de la release) (mais elles seront bien aux bonnes dates).