Je suis d’accord avec vous, et j’ai constaté que c’était l’une des complications techniques les plus importantes du « regroupement des plugins avec le cœur ». Nous avions quelques plugins bifurqués et il était très compliqué de les faire fonctionner sans supprimer le plugin regroupé.
Je ne pense pas que Jay suggère cela. Un plugin peut également remplacer des parties très spécifiques d’un autre plugin.
La meilleure approche serait de convaincre l’équipe que votre code mérite d’être fusionné dans le plugin officiel. Cela fonctionnera si votre modification est suffisamment générique ou flexible. Je vois que vous avez déjà fait une bifurcation et que vos modifications/ajouts sont assez propres. Peut-être que la chaîne de caractères codée en dur “Flagged” pourrait se trouver dans un fichier de traduction et si vous faites en sorte que :flagged soit faux par défaut, vous n’avez pas à modifier le gestionnaire d’événements d’origine avec un paramètre supplémentaire mais à part cela, cela semble valable. Si j’étais vous, je le mettrais à jour, j’ouvrirais une PR et j’en discuterais dans le sujet du plugin.
Si cette voie échoue, vous pourriez simplement créer un plugin qui remplace les trois fonctions que vous avez modifiées et ajoute le gestionnaire on(:reviewable_created).