-
Notifications
You must be signed in to change notification settings - Fork 29
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
Warnings de dependencias #1118
Comments
adopted-ember-addons/ember-paper#1202 Issue en ember paper sobre ember 4. Entonces tenemos que seguir por el plan del fork. Investiguemos por ahí: actualizarle las dependencias a paper. Quizás se pueden revisar los PR que hoy tiene ember-paper a ver si alguno soluciona alguno de los problemas. |
Quedan:
|
Al subir a la version v3.24 de ember, se deprecaron tryInvoke, getWithDefault y otras features de ember, las cuales son usadas por algunas de nuestras dependencias, particularmente ember-paper. Esto hace que se tiren muchos warnings molestos, especialmente al correr tests.
Una forma de resolver varios de estos warnings, es actualizando las dependencias de ember-paper. Para hacer esto podriamos hacer un fork de ember-paper (de nuevo), lo cual vendria bien porque la ultima version de ember-paper usa getWithDefault (deprecado), lo cual no se puede resolver subiendo versiones de dependencias.
Tambien podriamos subir la version de node de pilas bloques a gallium para poder usar el override en package.json para forzar que ember-paper use dependencias mas actualizadas. Esto lo intentamos asi nomas pero explota, asi que parece que hay que meterle un poco de cerebro para hacerlo. Aun asi, esto no resolveria lo de que ember-paper usa getWithDefault.
Tirar un PR a ember-paper no es una opcion porque esta muy muerto :).
Tambien habria que actualizar la dependencia ember-composable-helpers a la version 5.0.0, pero esta dependencia solo causa unos pocos warnings, relativo al total de la pared de texto gigante de warnings totales, asi que es algo menor.
Tambien hay otros warnings que no vimos en profundidad porque no pudimos resolver lo de ember-paper.
Sufrimiento.
The text was updated successfully, but these errors were encountered: