Update NPM packages #75
Conversation
|
Why do we need to update all of the packages? |
Just going on an update spree. Dependabot has been submitting a few PRs here and there so I figure it's best to keep packages up to date: #73 Also these alerts: https://github.com/actions/setup-python/network/alerts |
A few tricks here to help:
|
|
@brcrista figured it out, I followed the recommendations here: https://prettier.io/docs/en/options.html#end-of-line |
Sanity check to make sure everything works before going to master: https://github.com/konradpabjan/Testing2/runs/533477128?check_suite_focus=true
This should make the security warning go away that we're seeing.
I used
ncuto update all the packages (haven't found a better way to update all npm packages)https://stackoverflow.com/questions/16073603/how-do-i-update-each-dependency-in-package-json-to-the-latest-version
Also
npm run format-checkis green on my windows box, but it's failing here?!? looks like it's just line endings so that can be ignored, still curious as to why (I've got CRLF in vscode andcore.autocrlfset to true for git)?