MineWeb / MineWebCMS

🚀 A French Minecraft CMS since 2015 (used by +1k websites)
https://mineweb.org
GNU General Public License v3.0
93 stars 41 forks source link

New version : 1.14 #285

Closed nivcoo closed 2 years ago

StanByes commented 2 years ago

👍

jimmyr57000 commented 2 years ago

👍

MaximeMichaud commented 2 years ago

Gros pull 😯

ThomasFarineau commented 2 years ago

👍

jimmyr57000 commented 2 years ago

@MaximeMichaud Ajoute voir un 👍

MaximeMichaud commented 2 years ago

@MaximeMichaud Ajoute voir un 👍

Voulez-vous que je le fasse sachant que je ne devrais pas donner mon opinion puisque je n'ai rien testé, etc ?

jimmyr57000 commented 2 years ago

@MaximeMichaud Ajoute voir un 👍

Voulez-vous que je le fasse sachant que je ne devrais pas donner mon opinion puisque je n'ai rien testé, etc ?

Tu m'enerve !

nivcoo commented 2 years ago

@MaximeMichaud Ajoute voir un 👍

Voulez-vous que je le fasse sachant que je ne devrais pas donner mon opinion puisque je n'ai rien testé, etc ?

C'est juste que je ne peux pas approuver mon propre pull pour le merge aprĂšs donc le bot me bloque ;x

MaximeMichaud commented 2 years ago

👍

nivcoo commented 2 years ago

Bon beh le bot est officiellement cassé hein

MaximeMichaud commented 2 years ago

Bon beh le bot est officiellement cassé hein

Ouais, il me semblait puisque c'était 2 max 3 personnes de base je crois. Sinon, il a pas fonctionné depuis trop longtemps ? Haha

nivcoo commented 2 years ago

Bon beh le bot est officiellement cassé hein

Ouais, il me semblait puisque c'était 2 max 3 personnes de base je crois. Sinon, il a pas fonctionné depuis trop longtemps ? Haha

Si c'est un bot externe il doit ĂȘtre off mais c'est con de pas pouvoir juste approuver ses propre pull lol

nivcoo commented 2 years ago

Sinon je fais sans pull mais je trouvais ça mieux quand mĂȘme

Eywek commented 2 years ago

Tu dois pouvoir merge mtn, j’ai enlevĂ© les required checks. Bon par contre, c’est illisible autant de changements, surtout avec de l’Indentation

nivcoo commented 2 years ago

Tu dois pouvoir merge mtn, j’ai enlevĂ© les required checks. Bon par contre, c’est illisible autant de changements, surtout avec de l’Indentation

J'ai pourtant essayé de faire au maximum un commit par action, j'ai pas lié l'indent avec les commits de modifications, et je dois squash quand je merge ou je merge juste ? Je serais d'avis de juste merge pour garder l'historique ?

Eywek commented 2 years ago

Tu dois pouvoir merge mtn, j’ai enlevĂ© les required checks.

Bon par contre, c’est illisible autant de changements, surtout avec de l’Indentation

J'ai pourtant essayé de faire au maximum un commit par action, j'ai pas lié l'indent avec les commits de modifications, et je dois squash quand je merge ou je merge juste ? Je serais d'avis de juste merge pour garder l'historique ?

My bad j'ai pas regardé commits par commits, la vue de Github de base est pas pratique pour ça malheureusement Ouais vu que t'as fais pas mal de trucs autant merge

nivcoo commented 2 years ago

Tu dois pouvoir merge mtn, j’ai enlevĂ© les required checks.

Bon par contre, c’est illisible autant de changements, surtout avec de l’Indentation

J'ai pourtant essayé de faire au maximum un commit par action, j'ai pas lié l'indent avec les commits de modifications, et je dois squash quand je merge ou je merge juste ? Je serais d'avis de juste merge pour garder l'historique ?

My bad j'ai pas regardé commits par commits, la vue de Github de base est pas pratique pour ça malheureusement Ouais vu que t'as fais pas mal de trucs autant merge

Okay thx, justement j'avais pensĂ© aux indents, je merge, je publish et aprĂšs je release et avec StanByes je vais tester la maj pour ĂȘtre sur que ça fonctionne au moment de la maj (Car oui le systĂšme de maj est rĂ©glĂ© mais il y a certain cas ou si on modifie la table user ça bug donc peut ĂȘtre que sur d'autres table importante il y a aussi le bug).

nivcoo commented 2 years ago

Ah beh je peux pas merge simplement (github le permet pas https://pics.revolis.fr/2021-08/28/22-30_rbCf.png), je vais passer par le terminal

nivcoo commented 2 years ago

Tu dois pouvoir merge mtn, j’ai enlevĂ© les required checks.

Bon par contre, c’est illisible autant de changements, surtout avec de l’Indentation

J'ai pourtant essayé de faire au maximum un commit par action, j'ai pas lié l'indent avec les commits de modifications, et je dois squash quand je merge ou je merge juste ? Je serais d'avis de juste merge pour garder l'historique ?

My bad j'ai pas regardé commits par commits, la vue de Github de base est pas pratique pour ça malheureusement Ouais vu que t'as fais pas mal de trucs autant merge

Je peux pas push sur master, Et j'ai pas la perm de supprimer de tags non plus :/

remote: error: GH006: Protected branch update failed for refs/heads/master. remote: error: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information. To github.com:MineWeb/MineWebCMS.git ! [remote rejected] master -> master (protected branch hook declined) error: failed to push some refs to 'git@github.com:MineWeb/MineWebCMS.git'

Eywek commented 2 years ago

Tu dois pouvoir merge mtn, j’ai enlevĂ© les required checks.

Bon par contre, c’est illisible autant de changements, surtout avec de l’Indentation

J'ai pourtant essayé de faire au maximum un commit par action, j'ai pas lié l'indent avec les commits de modifications, et je dois squash quand je merge ou je merge juste ? Je serais d'avis de juste merge pour garder l'historique ?

My bad j'ai pas regardé commits par commits, la vue de Github de base est pas pratique pour ça malheureusement

Ouais vu que t'as fais pas mal de trucs autant merge

Je peux pas push sur master, Et j'ai pas la perm de supprimer de tags non plus :/

remote: error: GH006: Protected branch update failed for refs/heads/master.

remote: error: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information.

To github.com:MineWeb/MineWebCMS.git

! [remote rejected] master -> master (protected branch hook declined)

error: failed to push some refs to 'git@github.com:MineWeb/MineWebCMS.git'

J'ai unprotect la branch c'est bon Pour le tags normalement une fois que t'auras push le commit ça sera bon vu que le hash aura pas changé

nivcoo commented 2 years ago

Tu dois pouvoir merge mtn, j’ai enlevĂ© les required checks.

Bon par contre, c’est illisible autant de changements, surtout avec de l’Indentation

J'ai pourtant essayé de faire au maximum un commit par action, j'ai pas lié l'indent avec les commits de modifications, et je dois squash quand je merge ou je merge juste ? Je serais d'avis de juste merge pour garder l'historique ?

My bad j'ai pas regardé commits par commits, la vue de Github de base est pas pratique pour ça malheureusement

Ouais vu que t'as fais pas mal de trucs autant merge

Je peux pas push sur master, Et j'ai pas la perm de supprimer de tags non plus :/ remote: error: GH006: Protected branch update failed for refs/heads/master. remote: error: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information. To github.com:MineWeb/MineWebCMS.git ! [remote rejected] master -> master (protected branch hook declined) error: failed to push some refs to 'git@github.com:MineWeb/MineWebCMS.git'

J'ai unprotect la branch c'est bon Pour le tags normalement une fois que t'auras push le commit ça sera bon vu que le hash aura pas changé

Oui merci (Je parle surtout que le tag j'avais mis 1.14 et pas 1.14.0 car j'avais pas vu la forme de base donc il y a un tag inutile)