HisteriaMC / histeria-wiki

Wiki communautaire du serveur Minectraft Bedrock Histeria
https://histeria.fr
Apache License 2.0
14 stars 15 forks source link

Shield titre #203

Closed Tonmou8128 closed 7 months ago

Tonmou8128 commented 7 months ago

Fix #199 Il manquait un "d" au titre de la page Shield

dadodasyra commented 7 months ago

Il faudrais mettre a jours ton fork, tu as trop de commit qui ne concerne pas la PR. Regarde la fonction de synchronisation sur ta page de ta repository

il a déjà merge, les commits comptent pas vraiment de ce que je vois, en tout cas dans file changed y a rien

zelytra commented 7 months ago

Il faudrais mettre a jours ton fork, tu as trop de commit qui ne concerne pas la PR. Regarde la fonction de synchronisation sur ta page de ta repository

il a déjà merge, les commits comptent pas vraiment de ce que je vois, en tout cas dans file changed y a rien

Oui je sais, mais c'est pas propre et un rebase/ mise a jour du git est nécessaire

Tonmou8128 commented 7 months ago

J'avais synchronisé mon fork avant de créer la PR, pour les commits je vais voir pour les enlever, et il y a pourtant la page de Shield dans les files changed

zelytra commented 7 months ago

J'avais synchronisé mon fork avant de créer la PR, pour les commits je vais voir pour les enlever, et il y a pourtant la page de Shield dans les files changed

Oui oui, t'as modification est prise en compte mais il y a tout les commits non concerné qui pollue la PR. Rebase t'as branche depuis ton main doit être a jours avec le main de histeria/histeria-wiki https://docs.github.com/fr/get-started/using-git/about-git-rebase

Tonmou8128 commented 7 months ago

J'avais synchronisé mon fork avant de créer la PR, pour les commits je vais voir pour les enlever, et il y a pourtant la page de Shield dans les files changed

Oui oui, t'as modification est prise en compte mais il y a tout les commits non concerné qui pollue la PR. Rebase t'as branche depuis ton main doit être a jours avec le main de histeria/histeria-wiki https://docs.github.com/fr/get-started/using-git/about-git-rebase

J'ai cherché, le problème que je rencontre est que je n'ai pas accès à l'invite de commandes car je suis principalement sur mobile, et les guides utilisent uniquement cette méthode. Mais je me suis quand même débrouillé pour créer une nouvelle branche identique à main dans mon fork, qui n'a aucun commit. Je fais en effet toujours attention de synchroniser ma branche avant de faire une PR. Si cette méthode est la bonne ( ou pas mais il n'y a pas d'inconvénients ), alors est-ce que je dois fermer cette PR et en ouvrir une nouvelle ou faire une manipulation sur celle-ci ?

zelytra commented 7 months ago

Il est techniquement possible de modifier la branche pour résoudre les probs, si cela t'arrange de créer une nouvelle branche et donc une nouvelle PR tu peut