그 마음 충분히 이해합니다. 그리고 저도 “플러그인을 코어에 번들링하는” 방식의 가장 큰 기술적 복잡성 중 하나로 이 문제를 꼽고 있습니다. 포크한 플러그인이 몇 개 있었는데, 번들링된 플러그인을 제거하지 않고는让它们 작동시키는 것이 매우 복잡했습니다.
제가 보기엔 Jay는 그런 의도는 아닌 것 같습니다. 플러그인은 다른 플러그인의 매우 구체적인 부분만 오버라이드할 수도 있습니다.
가장 좋은 접근 방식은 여러분의 코드가 공식 플러그인에 병합될 가치가 있다는 점을 팀을 설득하는 것입니다. 수정 사항이 충분히 일반적이거나 유연하다면 이것이 가능할 것입니다. 이미 포크를 만들었고 변경/추가 사항이 꽤 깔끔해 보입니다. 아마 하드코딩된 “Flagged” 문자열을 번역 파일로 옮기고, :flagged의 기본값을 false로 설정하면 추가 파라미터를 통해 원래의 이벤트 핸들러를 수정할 필요가 없게 됩니다. 그 외에는 병합할 가치가 있어 보입니다. 제가 당신이라면 최신 버전으로 업데이트하고 PR을 열어 플러그인 토픽에서 이 문제를 논의할 것입니다.
그 경로가 실패한다면, 단순히 여러분이 변경한 그 세 가지 함수를 오버라이드하고 on(:reviewable_created) 핸들러를 추가하는 플러그인을 구축하면 됩니다.