Skip to content
This repository was archived by the owner on Aug 30, 2025. It is now read-only.

Conversation

Louis454545
Copy link
Contributor

@Louis454545 Louis454545 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
Member

Bonsoir non.

@Louis454545
Copy link
Contributor Author

pour fix #177

@Louis454545
Copy link
Contributor Author

Bonsoir non.

pour cette issue : #177

@oriionn
Copy link
Member

oriionn commented Sep 7, 2024

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

@oriionn oriionn closed this Sep 7, 2024
@Louis454545
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

@Louis454545
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
Member

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

@Louis454545
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
Member

oriionn commented Sep 7, 2024

J'ai vu, je rajoutais juste une info 💀

@Louis454545
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
Member

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
Member

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 💀

@Louis454545
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
Member

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

@Louis454545
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

@Louis454545
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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants