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

Regularizar dependencias npm #129

Closed
wants to merge 2 commits into from
Closed

Regularizar dependencias npm #129

wants to merge 2 commits into from

Conversation

luchosrock
Copy link

  • Se ajustan las dependencias para pasar los checks de vulnerabilidades de github 😸
  • Se corrige bug Hacer repo npm-compatible #107 (compatible con npm y yarn)

Revisé las dependencias mencionadas y practicamente todas tenían que ver con la versión de webpack que usa el paquete preact-cli. Hice el push a mi fork, habilité el dependency graph y los insights y no me aparecieron alertas.

Quedo atento a cualquier feedback ❤️

@enBonnet
Copy link
Contributor

enBonnet commented Dec 6, 2018

@cristofer-dev Tu tienes permisos para aprobar este pr?

@cristofer-dev
Copy link

@enBonnet todos deberiamos tener!

Sólo deben ir a
captura de pantalla de 2018-12-06 10-01-53

Revisar los cambios y luego en:
captura de pantalla de 2018-12-06 10-02-46

Aprobar o Solicitar cambios.

Para llegar a Master necesitas al menos que 1 apruebe los cambios. De momento nadie a hecho el review por eso el merge esta bloqueado.

@cristofer-dev
Copy link

Ha si... y el DEPLOY a netlify fallo, asi es que es probable que tambien falle en PROD ?
captura de pantalla de 2018-12-06 10-04-40

@luchosrock
Copy link
Author

Ha si... y el DEPLOY a netlify fallo, asi es que es probable que tambien falle en PROD ?
captura de pantalla de 2018-12-06 10-04-40

uhm, por eso será el bloqueo? Funciona en mi pc ahahahaah, voy a probar en otro equipo y te digo

@cristofer-dev
Copy link

Ambas, el boqueo es por que nadie ha aprobado y xq explot el deploy

@luchosrock luchosrock closed this Dec 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants