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

FIx pr #99

Merged
merged 1 commit into from
May 12, 2024
Merged

FIx pr #99

merged 1 commit into from
May 12, 2024

Conversation

Izocel
Copy link
Member

@Izocel Izocel commented May 12, 2024

No description provided.

@Izocel Izocel self-assigned this May 12, 2024
@Izocel
Copy link
Member Author

Izocel commented May 12, 2024

@CtrlAiDel tu me diras si c'est ok.
Si ca continue je vais même plus regardé tes PR et le repos sera private.

@Izocel Izocel requested a review from a team May 12, 2024 14:32
@CtrlAiDel
Copy link
Contributor

Oui c'est ok.

Au sujet du problème de PR, est-ce qu'on pourrait, si j'ai la même situation qui se reproduit, que je te paye et que on prenne le temps que j'utilise l'outil que tu as developper. Car en ce moment a part monter une liste ben basique en json, j'en fait pas plus.

Je ne comprends pas si j'ai mal gérer ou plûtot comment jai mal gérer mon pull et mon fetch.

Bref, si la prochaine fois est problèmatique avant de couper court au projet, est-ce possible de me revenir et qu'on regarde si un dernier court / reminder serait possible svp.?

@Izocel
Copy link
Member Author

Izocel commented May 12, 2024

Le tool ne peut pas t'aider pour la raison de la rejection de ta PR....
Ici tu avait un problème de branche pas aligné.

Oui c'est exactement ça:
"jai mal gérer mon pull et mon fetch"

Avant de 'déclencher' une PR il y a 2-3 vérifications minimales à faire et dans cet ordre:

  1. Dans le cas d'une fork il faut que celle-ci soit à jour avant. Ex: origine/dev merged-> fork/dev
  2. Ramasser le stock (fork ou origine) pour ta PR car elle peut avoir bougé. Ex: fork/dev merged-> pull/update
  3. Vérifier les files changes qui vont faire partie de la PR pour attraper un oubli, une erreur ou des modifications non désirées.

Merci

@Izocel Izocel merged commit 05b32e2 into dev May 12, 2024
2 checks passed
@CtrlAiDel
Copy link
Contributor

Merci, je m'envoi le tout par courriel et j'enregistre ça sur mon bureau.

Izocel added a commit that referenced this pull request May 12, 2024
@Izocel Izocel deleted the prFixes/98 branch May 12, 2024 20:49
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.

2 participants