cloudinary / cloudinary/cloudinary_gem

[Suggestion] Restructure source code to make integration easier

Aperta
#180 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
Ruby
Stelle
420
Fork
285
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia esaminando l'attuale struttura del codice sorgente della gem e la solida integrazione con Rails descritta nella issue. Valuta come separare il codice non specifico di Rails dal codice specifico di Rails, esponendo l'integrazione con Rails tramite lib/cloudinary/rails.rb. Il lavoro è completato quando il core può essere utilizzato con framework Ruby diversi da Rails senza dover rimuovere manualmente l'integrazione con Rails.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
rails, ruby
Ambito
backend
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
20/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.