[Suggestion] Restructure source code to make integration easier
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Facilidade para iniciantes
- 20/100
Direção de pesquisa
Comece revisando o layout atual do código-fonte da gem e a forte integração com Rails descrita na issue. Avalie como o código não específico do Rails poderia ser separado do código específico do Rails, com a integração com Rails exposta por meio de lib/cloudinary/rails.rb. Considera-se concluído quando o core puder ser usado por frameworks Ruby que não sejam Rails sem remover manualmente a integração com Rails.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Ruby
- Estrelas
- 420
- Forks
- 282
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de cloudinary/cloudinary_gem
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 55/100
cloudinary/cloudinary_gem#597 · 1 comentário ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 52/100
cloudinary/cloudinary_gem#591 · 3 comentários ·
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 45/100
cloudinary/cloudinary_gem#582 · 1 comentário ·
-
404's when fetching .eml files (from Action Mailbox)Talvez livre de novo @rnamba-cloudinary assumiu há 400 dias e não há nenhum pull request aberto. Aberta
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 30/100
cloudinary/cloudinary_gem#581 · 1 comentário ·
-
BlobKey, MirrorService and Sidekiq 8 are incompatibleTalvez livre de novo @const-cloudinary assumiu há 541 dias e não há nenhum pull request aberto. Aberta
cloudinary/cloudinary_gem#573 · 3 comentários · 1 responsável ·
Todas as issues de cloudinary/cloudinary_gem
Issues semelhantes
-
bug
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
yegor256/mailanes#361 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Property accounts have no notes fieldTalvez já em andamento @arnaurodondev assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
we-promise/sure#5057 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
metacall/core#964 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
area: ci benchmark build developer-experience difficulty: easy dx enhancement github-actions good first issue hacktoberfest help wanted performance quality rank: 8 kyu ruby tech: ruby
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 82/100
flisk-sh/flisk-labs#22 ·
-
1.severity: security
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 62/100
Mantenedores costumam responder em até 1 dia