Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Crab DI (Koin wrapper) #3020

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

leandrotemperoni
Copy link
Contributor

@leandrotemperoni leandrotemperoni commented Nov 1, 2024

Descripción

Cual es la necesidad de agregar este cambio ?

  • Luego de agregar Koin en este PR, agregamos transitivity: false nuevamente
  • Por otro lado, sumamos Crab-DI, la lib que funcionará como wrapper de Koin
Seleccione el tipo de su dependencia.
Tipo Descripción
  • Interna
Libreria/modulo desarrollado in-house en base al ecosistema de Meli.
  • Externa
Libreria desarrollada por un externo a Meli. (Google, Airbnb, otros). Requiere Ticket de Shield.
ETA
  • Leí el contenido de este Pull Request y acepto que recibiré seguimiento del mismo una vez su contenido y el de los Tickets solicitados esté completo.
Sección obligatoria para Dependencias Externas

ℹ️ SHIELD Ticket

Link de los Tickets
Descripción Android iOS
Dependencia Externa con Artefacto en Nexus Upload an artifact Android to Repository Upload an artifact iOS to Repository
Dependencia Externa External Dependency Analysis Android External Dependency Analysis iOS
Agregue el Ticket de Shield.

Dependency Shield Ticket

Información

Seleccione las Aplicaciones impactadas por esta dependencia.

Mercado Libre Mercado Pago SmartPOS Alicia: Flex / Logistics WMS Meli Store

ℹ️ Seleccione el tipo de dependencia.

Frontend Cross

ℹ️ Mi dependencia tienes un uso controlado ?

Si No

Esta dependencia agrega Artefactos (Jar, Arr, Framework, ...) o Dependencias Externos?

En caso de ser Si, agregue el Link de Nexus - Nexus Link

Si No

@Cavigliano
Copy link
Collaborator

@leandrotemperoni como estas? necesitamos que completen toda la info de la descripción del PR y sus checks. Gracias!

@leandrotemperoni
Copy link
Contributor Author

@Cavigliano Gracias por revisar el PR. Deje completo los checks, de todos modos debemos esperar a que @lcarboneti confirme si se pudo solucionar el issue con transitivity. Dejamos el PR en draft tal vez?

@Cavigliano Cavigliano marked this pull request as draft November 4, 2024 17:06
@lcarboneti
Copy link
Collaborator

@leandrotemperoni tenemos un problema con la generacion dedependencias y subidas a nexus por el cambio reciente de la gente que gestiona el tema de credenciales via FURY CLI . por esto no he podido probar la solucion. en cuanto est este resuelto podre darles mas datos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants