cloudinary / cloudinary/cloudinary_gem
[Suggestion] Restructure source code to make integration easier
- 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