Capisco, e anche io ho riscontrato che questa è una delle complicazioni tecniche più significative del “raggruppare i plugin con il core”. Avevamo alcuni plugin biforcuti ed era molto complicato farli funzionare senza rimuovere il plugin raggruppato.
Non credo che Jay stia suggerendo questo. Un plugin può anche sovrascrivere parti molto specifiche di un altro plugin.
L’approccio migliore sarebbe convincere il team che il tuo codice merita di essere unito al plugin ufficiale. Funzionerà se la tua modifica è generica o sufficientemente flessibile. Vedo che hai già creato un fork e le tue modifiche/aggiunte sono piuttosto pulite. Forse la stringa “Flagged” codificata potrebbe essere in un file di traduzione e se fai in modo che :flagged sia false per impostazione predefinita, non devi modificare l’handler di evento originale con un parametro aggiuntivo, ma a parte questo, sembra degno. Se fossi in te, lo aggiornerei, aprirei una Pull Request (PR) e ne discuterei nell’argomento del plugin.
Se questa strada fallisce, potresti semplicemente creare un plugin che sovrascriva le tre funzioni che hai modificato e aggiunga l’handler on(:reviewable_created).