Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add ConditionalCommitAuthorizedEventLog #593

Merged
merged 3 commits into from
Oct 19, 2023

Conversation

albertfolch-redeemeum
Copy link
Collaborator

No description provided.

@albertfolch-redeemeum albertfolch-redeemeum linked an issue Oct 18, 2023 that may be closed by this pull request
@albertfolch-redeemeum albertfolch-redeemeum force-pushed the 591-index-conditionalcommitauthorized branch 3 times, most recently from ada2b5a to eccca7c Compare October 18, 2023 15:51
@albertfolch-redeemeum albertfolch-redeemeum force-pushed the 591-index-conditionalcommitauthorized branch from 5033765 to e13a8b3 Compare October 18, 2023 16:18
@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Merging #593 (30a5361) into main (116640e) will decrease coverage by 30.31%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main     #593       +/-   ##
===========================================
- Coverage   83.77%   53.46%   -30.31%     
===========================================
  Files         133      124        -9     
  Lines        3925     3763      -162     
  Branches      677      641       -36     
===========================================
- Hits         3288     2012     -1276     
- Misses        637     1751     +1114     
Flag Coverage Δ
common 84.93% <ø> (ø)
core-sdk 51.59% <100.00%> (+0.19%) ⬆️
e2e ?
eth-connect-sdk 18.30% <ø> (ø)
ethers-sdk 67.63% <ø> (ø)
ipfs-storage ?
metadata ?
unittests 53.46% <100.00%> (-1.67%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
packages/core-sdk/src/subgraph.ts 88.48% <100.00%> (-5.13%) ⬇️

... and 59 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@albertfolch-redeemeum albertfolch-redeemeum merged commit c799b27 into main Oct 19, 2023
6 of 7 checks passed
@albertfolch-redeemeum albertfolch-redeemeum deleted the 591-index-conditionalcommitauthorized branch October 19, 2023 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

index ConditionalCommitAuthorized
2 participants