Le 22/06/2020 à 09:35, Cerdic a écrit :
J’ai donc envoyé un revert complet des 4 commits de JLuc que tout soit bien clair, et recommit le seul changement de cette salve, qui était une URL dans le PHPDoc.

Merci Cerdic !

JLuc, ça serait quand même malin de tester un peu avant d’envoyer ce genre de gros diff, et si tu n’est pas certain ET que tu as besoin de commit pour déployer et tester, le mieux c’est de faire une branche : comme ça en ligne tu checkout sur cette branche, tu teste, tu debug, et quand ça marche tu peux faire une Pull Request, ou cherry-picker pour envoyer sur le master, ou demander à quelqu’un de le faire.

Oui je suis en environnement de dev temporairement bancal
et visiblement je ferais mieux de m'abstenir tant que c'est pas upgradé 
consolidé.

Pour info du coup j'avais commit via l'UI gitea
et c'est donc probablement ça qui a ajouté les espaces,
alors pourtant que ça ne s'était pas produit toutes les fois précédentes où 
j'ai commit comme ça
(alors je pige pas le pourquoi).

Et donc, instant documentation :
Comme indiqué par Maieul,

Non ya pas eu indiqué à ma connaissance.

C'est une commandes très basique finalement pour revert.
Encore faut il savoir que c'est aussi simple car c'est souvent compliqué de 
corriger un truc déjà pushé.

re-merci
JL

pour générer un commit qui revert un autre commit il faut faire
git revert xxxxxx > (avec le numéro du commit à revert )


Pour voir le diff entre 2 commits c'est
git diff abcdef..123456
(la référence du premier commit, deux petits points et la référence du second 
commit).

Et si on ajoute l’option -w ça ignore les changements qui ne sont que des 
espaces.
Dans notre cas ça donnait :

$ git diff -w e350597..1900248
diff --git a/spip_bonux_options.php b/spip_bonux_options.php
index 5d83500..b2c6b6f 100644
--- a/spip_bonux_options.php
+++ b/spip_bonux_options.php
@@ -202,7 +202,7 @@ if (!function_exists('text_truncate')) {
  * @param array $options An array of html attributes and options.
  * @return string Trimmed string.
  * @access public
-* @link http://book.cakephp.org/view/1469/Text#truncate-1625
+ * @link https://api.cakephp.org/4.0/class-Cake.Utility.Text.html#truncate
  */
  function text_truncate($text, $length = 100, $options = array()) {
  $default = array(



--
Cédric
Le 21 juin 2020 à 21:58 +0200, JLuc <j...@no-log.org>, a écrit :
Le 21/06/2020 à 21:21, Eric Lupinacci a écrit :> Je viens de réessayer :> 
Warning: Cannot modify header information -
headers already sent by (output started at >
/Users/eric/Sites/ZONEGIT/gitea/spip-contrib-extensions/spip-bonux/spip_bonux_options.php:1)
 in >
/Users/eric/Sites/SPIP/ecrire/inc/actions.php on line /141/> En plus 
l’indentation a changé et d’ailleurs elle est
foireuse sous phpstorm.> Donc tant qu’à faire faut revenir vraiment à la 
version précédente du fichier svp.
OK
J'ai pas eu de réponse pour réverter
alors je vais commiter une nouvelle couche

JLuc

_______________________________________________
liste: https://listes.rezo.net/mailman/listinfo/spip-dev
doc: https://www.spip.net/
dev: https://core.spip.net/
irc://irc.freenode.net/spip



_______________________________________________
liste: https://listes.rezo.net/mailman/listinfo/spip-dev
doc: https://www.spip.net/
dev: https://core.spip.net/
irc://irc.freenode.net/spip

Répondre à