Update ffcommon, ffsigner, ffevmconnect #1579
Conversation
Signed-off-by: Matthew Whitehead <matthew1001@gmail.com>
| "sha": "11a782c778227d198c0bff2fc94ad7e52ceea5a9994728ba05e781541680c32d" | ||
| }, | ||
| "fabconnect": { | ||
| "image": "ghcr.io/hyperledger/firefly-fabconnect", |
There was a problem hiding this comment.
Probably need to update common here? I bet it has the same problem with JSON Numbers
There was a problem hiding this comment.
I don't think common is in the manifest list?
EnriqueL8
left a comment
EnriqueL8
left a comment
There was a problem hiding this comment.
Don't we need to update FabConnect and Tezos Connect to latest transaction manager as well?
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1579 +/- ##
=======================================
Coverage 99.75% 99.75%
=======================================
Files 338 338
Lines 24223 24223
=======================================
Hits 24164 24164
Misses 58 58
Partials 1 1 ☔ View full report in Codecov by Sentry. |
Yes to tezos - I'll do that now. Fabconnect doesn't pull in |
|
Going to update the manifest with Tezos and Fabric at another point when we release a new rc |
2 participants
This pulls in a fix in
firefly-commonrelated to large number handling, and updates the other related dependencies and manifest list