-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat : ca lance en pr #183
Conversation
Bonsoir non. |
pour fix #177 |
pour cette issue : #177 |
Ton action build et PUBLIE les builds sur les stores… |
non, ta pas bien regarder |
ça publie des artifacts pour chaque PR ce n'est donc pas acceptable et ça fait 15 fois qu'on le dit |
c'est bon ? |
J'ai vu, je rajoutais juste une info 💀 |
je crois que tu peux modifier le temps que les artifacts existent |
Par contre, j'ai une petite demande, je ne sais pas si c'est possible, mais est-ce possible d'éviter de lancer le build que s'il y a une modification dans le code de l'app, pour éviter de lancer l'actions dans une PR qui modifie les fichiers hors app (Comme celle ci par exemple) Bon c'est un petit détails, mais sinon ça m'a l'air nickel |
Par contre, là avec ta PR j'ai un petit doute sur le fait que la signature des apps pose un petit problème dans la GitHub Actions 💀 |
normal c'est une securité github, sinon on pourrait divulguer les certificats... |
Oui, du coup, il faudrait peut-être voir pour éviter de signer les certificats quand c'est un build de PR (je sais pas pas si on peut build des ipa sans certificat 💀 ) |
askip si |
les artifacts seront tjr récupérable dans la catégorie Actions, je ne comprends toujours pas pourquoi un tel acharnement pour une feature aussi useless que simplement 3 cliques et une commande dans un IDE |
Wesh tu fais comment |
🚀 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
ca lance en pr
Allez dans les paramètres de votre dépôt
Sélectionnez "Branches" dans le menu latéral
Cliquez sur "Add rule" sous "Branch protection rules"
Choisissez la branche à protéger
Activez les options suivantes :
"Require pull request reviews before merging"
"Require status checks to pass before merging"