cloudinary / cloudinary/cloudinary_gem
[Suggestion] Restructure source code to make integration easier
- Vorherrschende Sprache
- Ruby
- Sterne
- 420
- Forks
- 285
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Hi team
Firstly, thanks for a good service. We are using cloudinary gem at Envato and it has been smooth.
When I look into the source code of this gem, I find that there was a strong Rails integration (which is fine), however I think this gem can/should be used by other non-Rails framework too (or even pure Ruby). So I am wondering what do you think about splitting the non-Rails code into a core component and having a separate `lib/cloudinary/rails.rb` to include the core + rails integration. I believe this would allow other frameworks to develop integration with cloudinary without having to manually remove the rails integration part.
I am happy to provide PR if your team think it aligns with your product roadmap.
Beitragsleitfaden
Rechercherichtung
Beginne damit, das aktuelle Quelllayout des Gems und die in der Issue beschriebene starke Rails-Integration zu überprüfen. Bewerte, wie der Nicht-Rails-Code vom Rails-spezifischen Code getrennt werden könnte, wobei die Rails-Integration über lib/cloudinary/rails.rb bereitgestellt wird. Als abgeschlossen gilt die Aufgabe, wenn der Kern von Nicht-Rails-Ruby-Frameworks verwendet werden kann, ohne die Rails-Integration manuell entfernen zu müssen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rails, ruby
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 20/100