Package updates for security

Co-Epi / mobile-app

Initial ReactNative code for CoEpi app for both iOS and Android - this repo is no longer being used, please see app-ios and app-android repo's for latest native code versions

MIT License

25 stars 9 forks source link

Package updates for security #12

Closed mribbons closed 4 years ago

mribbons commented 4 years ago

minimist@^1.2.3, https://snyk.io/vuln/SNYK-JS-MINIMIST-559764 jest@^25.1.0, as per npm audit suggestion

scottleibrand commented 4 years ago

Just to confirm: the package-lock.json changes are also intentional?

mribbons commented 4 years ago

@scottleibrand That was the result of running npm install.

If we aren't using yarn, perhaps yarn.lock can be removed?

scottleibrand commented 4 years ago

I'm not an experienced developer myself, so I'm fine with both. Is this ready to merge now from your perspective?

mribbons commented 4 years ago

@scottleibrand good to go. re yarn.lock: yarn recommends against using other package managers concurrently, but yarn is much faster than npm.