search

hyperledger / anoncreds-rs

anoncreds-rs
https://wiki.hyperledger.org/display/anoncreds
Apache License 2.0
74 stars 55 forks source link

Deal with snyk notifications for react-native -- by updating to latest of all dependencies #328

Closed swcurran closed 9 months ago

swcurran commented 9 months ago

In making this PR, I looked up the latest versions of each of the called out in the Snyk report and used that. We'll see how that goes.

swcurran commented 9 months ago

@berendsliedrecht — can you please review this PR. Please note that I didn’t run any tests on this — I just updated the three components in the two package.json files to their most recent versions per NPM. I did not verify that the updated dependencies won’t break anything. I know — bad way to do things, but I don’t really know how to run the react-native wrapper/example.

Thanks!

berendsliedrecht commented 9 months ago

@swcurran could you delete the package-lock.json? It is created when running npm install, but we use pnpm only.

swcurran commented 9 months ago

Done — thanks!

swcurran commented 9 months ago

@berendsliedrecht — bump -- looks like this is ready.

swcurran commented 9 months ago

@andrewwhitehead — the post merge Lint task failed — https://github.com/hyperledger/anoncreds-rs/actions/runs/8031489343/job/21939863984. Could you please take a look and see what is to be done? Is it an issue or sunspots?