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

chore: simplification du remplissage des issues et des pull requests + nouveaux workflows + fixs extentions de fichiers incorrects #449

Open
wants to merge 61 commits into
base: main
Choose a base branch
from

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Dec 13, 2024

✨ Nouvelle Pull Request

Merci de contribuer à l'amélioration de Papillon !

Tu te poses des questions sur les pull requests (PR) ? Une documentation a spécialement été crée ici => https://gitbook.getpapillon.xyz/organisation/outils-internes/github

Avant toute chose...

Pour nous aider à tester ta PR, merci de cocher une des cases suivantes (en rajoutant un x dans les crochets) :

Type de pull request :

  • Breaking change (des modifications avec un impact sur les fonctionnalités actuelles)
    • J'ai fait une build de Papillon pour m'assurer que je n'ai rien cassé
  • Feat (ajoute une amélioration/nouveauté)
    • Je m'assure que j'utilise le langage informel (tutoiement)
  • Fix (permet de corriger un bug)
  • Chore (des modifications en dehors du dossier src)
  • Styles (change/ajoute du style)

Résumé des changements effectués

Note

La demande d'intégrer un workflow pour générer un qr code pour tester la pr directement n'est pas possible car Expo 52 est obligatoire ! Et Papillon n'est pour le moment pas compatible

  • Fix des extentions .yaml => .yml pour le bon fonctionnement de GitHub
  • Refonte du fichier PULL_REQUEST_TEMPLATE.md pour plus de simplicité lors de la création d'une PR
  • Mise à jour du pré remplissage des issues pour plus de simplification et Papillon vers la dernière version
  • Le workflow checks.yml a été mis à jour afin de prévenir directement les erreurs sur la conversation de la PR ! (et utilisation de npm plutôt que pnpm)
  • Le workflow start_build.yml a été mis à jour pour faire automatiquement une release sur Github et générer des apk par architecture

Capture(s) d'écran (pour rendre le test de ta PR rapide)

il y a pas besoin

Issue(s) en rapport

Note

Cette section permet de "linker" des issues à ta PR. Cela signifie qu'une fois ta PR mergée, les issues listées ci-dessous seront automatiquement fermées

S'il y en a plusieurs, continuer à les lister

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 13, 2024

@ecnivtwelve tu peux autoriser l'écriture des actions dans le repo pour que mon workflow fonctionne ?
c'est pour qu'il poste un message lorsqu'il y a une erreur TypeScript/ESLint

@Kgeek33 Kgeek33 mentioned this pull request Dec 15, 2024
9 tasks
@Kgeek33 Kgeek33 requested a review from Bulgus January 6, 2025 21:25
@Kgeek33 Kgeek33 requested a review from tryon-dev as a code owner January 18, 2025 00:15
@Bulgus
Copy link
Contributor

Bulgus commented Jan 19, 2025

Peut être pas la bonne PR, mais y'a des doublons # Expo dans le gitignore, si ça peut être clean au passage ^^

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 19, 2025

ok je vais en profiter pour fix ça ^^

@Gabriel29306
Copy link
Contributor

Up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
7 participants