chore(lib): update dependency next / xlsx #3877
Conversation
admin/package.json
Outdated
| @@ -68,7 +68,7 @@ | |||
| "snu-lib": "*", | |||
| "styled-components": "5.3.11", | |||
| "validator": "13.7.0", | |||
| "xlsx": "0.17.4" | |||
| "xlsx": "^0.18.5" | |||
There was a problem hiding this comment.
Que penses tu de laisser une version spécifique ? Pour éviter les régressions en cas d'update de la lib' ou de ses dépendances ?
There was a problem hiding this comment.
le lock file empêche que ça bouge.
perso je préfère garder les versions "compatible" pour les auto update afin de prendre les patch de sécurités et les améliorations mineurs (le jour ou on auto-upgrade, ce qui n'arrive presque pas).
@Johannbr on peux partir sur ~0.18.5 si tu préfère pour ne prendre que les patchs/fix.
There was a problem hiding this comment.
depuis, j'ai vérifié que dans la ci dans les tests on. a bien un npm ci qui est réalisé, donc c'est bon pour moi.
achorein
force-pushed
the
chore/update-dependencies-next-xlsx
branch
from
cab2b62 to
61267b4
Compare
achorein
force-pushed
the
chore/update-dependencies-next-xlsx
branch
from
61267b4 to
015c6f3
Compare
|
Code Climate has analyzed commit 015c6f3 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 49.7% (0.0% change). View more on Code Climate. |
* chore(lib): update dependency next / xlsx * update xlsx version
Ticket / Issue