Hello,

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.

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.

Et donc, instant documentation :

Comme indiqué par Maieul, 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 à