Skip to content
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

Closed
wants to merge 4 commits into from
Closed

Conversation

Louis-htmlcss
Copy link
Contributor

@Louis-htmlcss Louis-htmlcss commented Sep 7, 2024

🚀 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].

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

ca lance en pr

  1. Configurez les "branch protection rules" dans votre dépôt GitHub :
    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"

@tryon-dev
Copy link
Contributor

Bonsoir non.

@Louis-htmlcss
Copy link
Contributor Author

pour fix #177

@Louis-htmlcss
Copy link
Contributor Author

Bonsoir non.

pour cette issue : #177

@oriionn
Copy link
Collaborator

oriionn commented Sep 7, 2024

Ton action build et PUBLIE les builds sur les stores…
Sans commentaire.

@oriionn oriionn closed this Sep 7, 2024
@Louis-htmlcss
Copy link
Contributor Author

Ton action build et PUBLIE les builds sur les stores… Sans commentaire.

non, ta pas bien regarder

@oriionn oriionn reopened this Sep 7, 2024
@yannouuuu
Copy link
Member

Ton action build et PUBLIE les builds sur les stores… Sans commentaire.

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

@Louis-htmlcss
Copy link
Contributor Author

Ton action build et PUBLIE les builds sur les stores… Sans commentaire.

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 ?

@oriionn
Copy link
Collaborator

oriionn commented Sep 7, 2024

Ton action build et PUBLIE les builds sur les stores… Sans commentaire.

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

Sans oublier qu'il y a apparemment une limite sur le stockage d'artifacts 💀
image

@Louis-htmlcss
Copy link
Contributor Author

Ton action build et PUBLIE les builds sur les stores… Sans commentaire.

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

Sans oublier qu'il y a apparemment une limite sur le stockage d'artifacts 💀 image

si ta pas vu je viens de l'enlever

@oriionn
Copy link
Collaborator

oriionn commented Sep 7, 2024

J'ai vu, je rajoutais juste une info 💀

@Louis-htmlcss
Copy link
Contributor Author

J'ai vu, je rajoutais juste une info 💀

je crois que tu peux modifier le temps que les artifacts existent

@oriionn
Copy link
Collaborator

oriionn commented Sep 7, 2024

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

@oriionn
Copy link
Collaborator

oriionn commented Sep 7, 2024

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 💀

@Louis-htmlcss
Copy link
Contributor Author

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

@oriionn
Copy link
Collaborator

oriionn commented Sep 7, 2024

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 💀 )

@Louis-htmlcss
Copy link
Contributor Author

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

@tryon-dev tryon-dev closed this Sep 7, 2024
@yannouuuu
Copy link
Member

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

@Louis-htmlcss
Copy link
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants