[Suggestion] Restructure source code to make integration easier

Ouverte
#180 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
5/5
Temps estimé
Plus d'une semaine
Accessibilité débutants
20/100
Type d'issue
Refactorisation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
rails, ruby
Domaine
backend

Piste de recherche

Commencez par examiner l’organisation actuelle du code source du gem et la forte intégration à Rails décrite dans l’issue. Évaluez comment le code non spécifique à Rails pourrait être séparé du code spécifique à Rails, l’intégration à Rails étant exposée via lib/cloudinary/rails.rb. Le travail est considéré comme terminé lorsque le cœur peut être utilisé avec des frameworks Ruby autres que Rails sans devoir supprimer manuellement l’intégration à Rails.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Ruby
Étoiles
420
Forks
282
Métriques de merge des PR
Aucune PR mergée en 30 j

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de cloudinary/cloudinary_gem

Toutes les issues de cloudinary/cloudinary_gem

Issues similaires

Plus d'issues Ruby

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.