cloudinary / cloudinary/cloudinary_gem

[Suggestion] Restructure source code to make integration easier

Offen
#180 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.