[Suggestion] Restructure source code to make integration easier

Aperta
#180 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

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

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.

Descrizione

enhancement

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.

Lingua principale
Ruby
Stelle
420
Fork
282
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di cloudinary/cloudinary_gem

Tutte le issue di cloudinary/cloudinary_gem

Issue simili

Altre issue su Ruby

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.