-
Notifications
You must be signed in to change notification settings - Fork 5
feat: add multiple new certificate events test #302
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #302 +/- ##
==========================================
- Coverage 58.22% 58.05% -0.17%
==========================================
Files 230 230
Lines 11997 11997
==========================================
- Hits 6985 6965 -20
- Misses 5012 5032 +20
☔ View full report in Codecov by Sentry. |
e1cf7f7
to
e9f796c
Compare
Note: Polygon Edge related PR must be merged for this test to work |
Oh. Hmm. How can I run this PR with that PR applied? |
@atanmarko Is this the reason CI is failing? |
I had some additional fixes |
You can checkout
Note: GITHUB_TOKEN may not be needed anymore, it was needed when some of our Golang dependencies were private repositories Then run topos tests:
|
Put in a draft mode until Polygon Edge fix is merged |
3a8db4a
to
bd815cf
Compare
bd815cf
to
944347b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm modulo @Freyskeyd's comments. Left a few nitpicks to fix too.
944347b
to
3d47477
Compare
Description
Add test with multiple send token events
Fixes TP-691
PR Checklist: