add changes to insert 2 hooks : event_pre_section & event_post_section

[TractoBoy]

EN : It allows me to simulate Google Reader's behavior as indicated in issue #87 in a plugin.

FR : Cela me permet de simuler le comportement de Google Reader comme indiqué dans le problème #87 dans un plugin.

Lien / Link : https://github.com/TractoBoy/PluginGleeder
(Vous aurez besoin de la branche "fix_new_sections_hooks" / You'll need the "fix_new_sections_hooks" branch )
Etat initial :

Etat après clic :

PS : Mon plugin est vraiment sommaire, il faudrait l'étoffer avec du lazy loading, j'ai vu qu'après coup que vous incluiez le script de JQuery.

[Sbgodin]

@tous, serait-il possible de faire les pull request (PR) depuis dev plutôt que master ? Sauf erreur, c'est dédié à des correctifs urgents, qui doivent être récupérés aussi dans dev. Actuellement, on ne peut pas profiter des fonctionnalités ajoutées parce qu'elles sont dans la branche master. Et ça conflicte !

Normalement, il faudrait développer à partir de dev, pour vérifier la compatibilité avec les développements en cours et surtout faire le PR depuis la dev plutôt que la prod. En toute logique, et sauf patch urgent, il faudrait refuser les PR provenant de master.

Parce que là le cherry-pick ou le merge aboutissent à un conflit !

Deux solutions :

• Corriger au moment de la fusion master dans `dev. C'est crade, mais ça marche. Il vaut mieux que ce soit le développeur concerné qui se charge d'arbitrer.
• Annuler les commits 4f8225b et c4b6df2 pour revenir au d9bf2fa. Refaire la modification des hooks depuis la dev et faire le PR dans le bon sens : depuis dev vers dev.

En tout cas il faut y remédier de suite, sous peine de rendre les prochaines fusions difficile. Dans le pire des cas, je peux m'en occuper seul, pour la beauté de nos branches^^

@ldleman @TractoBoy ?

[Sbgodin]

J'ai testé la fusion. Ça rate parce que le corps de l'article est dans son propre fichier dans dev alors qu'il est dans l'index dans master. N'étant pas expert là-dedans, je peux difficilement déplacer les deux event...

Je suggère donc urgemment que :

• J'annule la dernière fusion (4f8225b).
• @TractoBoy (ou quelqu'un d'autre) fasse un branche depuis idleman/dev
• La requête de fusion aille bien depuis la branche dédiée vers idleman/dev

Me contacter au besoin. @ldleman @TractoBoy ?

[TractoBoy]

Bonsoir,

Concrètement, je n'ai aucune idée de votre problème. J'avais fait cette
suggestion il y a 7 mois, et depuis je me suis tourné vers un autre
agrégateur de flux.
Il me semble qu'à cette époque, le développement était encore peu
participatif, donc les branches dev & co n'existaient pas.
Mon commit étant simplement composé de deux ajouts de lignes, cela vaut-il
le coup de faire des rollbacks / new branch / refusion ?

La, j'avouerais ne pas avoir envie de retélécharger mon dépôt, faire les
manips demandées (je manie assez mal Git qui plus est), etc.
Le tout étant demandé sans bonjour ni merde, ça n'aide pas.

Bonne soirée,

TBoy.

2013/12/19 Sbgodin [email protected]

J'ai testé la fusion. Ça rate parce que le corps de l'article est dans son
propre fichier dans dev alors qu'il est dans l'index dans master. N'étant
pas expert là-dedans, je peux difficilement déplacer les deux event...

Je suggère donc urgemment que :

• J'annule la dernière fusion (4f8225bhttps://github.com/ldleman/Leed/commit/4f8225b39abf0b8b9386dac9cd707a4b06cf49ad
  ).
• @TractoBoy https://github.com/TractoBoy (ou quelqu'un d'autre)
  fasse un branche depuis idleman/dev
• La requête de fusion aille bien depuis la branche dédiée vers
  idleman/dev

Me contacter au besoin. @ldleman https://github.com/ldleman @TractoBoyhttps://github.com/TractoBoy?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/164#issuecomment-30948418
.

[Sbgodin]

Bonsoir @TractoBoy,

Je confirme que je n'ai rien à redire concernant ton patch. Il a l'air de faire ce qu'il faut, et tu l'as effectivement fait à l'époque ou master servait de dev. Ton patch a été fusionné 7 mois plus tard alors que la branche de développement avait changé de nom.

Je comprends que tu n'as pas envie de faire la suite, surtout après si longtemps. Et le maniement de Git, n'aide pas ! Je préfère largement Mercurial sur ce plan. Pas de soucis donc à ce qu'on prenne la suite, c'est comme ça que ça marche.

Actuellement, master conflicte avec dev. Cela peut freiner les développements ultérieurs en complexifiant un peu plus les fusions. C'est pour cela que je préfère traiter l'affaire au plus vite.

N'étant pas expert des plugins, je me risquerait pas à simplement déplacer les deux lignes de code concernées.

[cobalt74]

@Sbgodin je vais m'en occuper.

[cobalt74]

@Sbgodin j'ai annulé le commit sur la branche master et reporté en dev la demande.

[Sbgodin]

Merci ! Vu que j'avais remanié les retours chariots, la fusion de dev dans master coince dans action.php, mais il suffira d'imposer le action.php venant de dev. Un problème de moins ;-)