chore: upgrade ethers and did-provider-ethr for sepolia support

decentralized-identity / veramo

A JavaScript Framework for Verifiable Data

https://veramo.io

Apache License 2.0

414 stars 130 forks source link

chore: upgrade ethers and did-provider-ethr for sepolia support #1306

Closed strumswell closed 6 months ago

strumswell commented 6 months ago

What issue is this PR fixing

n/a

What is being changed

This PR bumps the version of ethr-did in the did-provider-ethr to add support for did:ethr:sepolia. I am also bumping ethers across the board for compatibility with the new ethr-did.

Quality

Check all that apply:

[X] I want these changes to be integrated
[X] I successfully ran pnpm i, pnpm build, pnpm test, pnpm test:browser locally.
[X] I allow my PR to be updated by the reviewers (to speed up the review process).
[ ] I added unit tests.
[X] I added integration tests.
[ ] I did not add automated tests because _____, and I am aware that a PR without tests will likely get rejected.

Details

n/a

mirceanis commented 6 months ago

looks good, but we should update ethers to the same version in all packages that have it: did-provider-ion did-provider-key did-provider-pkh key-manager kms-local utils

strumswell commented 6 months ago

Thanks for your feedback. Completely forgot about the other packages. Should be on the same version now across the board.

codecov-commenter commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2157e70) 85.47% compared to head (94a78b5) 85.47%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## next #1306 +/- ## ======================================= Coverage 85.47% 85.47% ======================================= Files 170 170 Lines 18950 18950 Branches 2115 2115 ======================================= Hits 16197 16197 Misses 2753 2753 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.