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

fix CI #13

Merged
merged 2 commits into from
Dec 8, 2023
Merged

fix CI #13

merged 2 commits into from
Dec 8, 2023

Conversation

zeroXbrock
Copy link
Member

@zeroXbrock zeroXbrock commented Dec 8, 2023

this fixes most of the failing actions; all the ones caused by our code, anyways. This branch has failing tests from optimism. Might need to merge in the latest from upstream to fix, but that can happen in another PR.

Copy link
Contributor

github-actions bot commented Dec 8, 2023

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
viem (esm) 57.93 KB (0%) 1.2 s (0%) 68 ms (-13.03% 🔽) 1.3 s
viem (cjs) 76.48 KB (0%) 1.6 s (0%) 155 ms (-10.85% 🔽) 1.7 s
viem (minimal surface - tree-shaking) 3.97 KB (0%) 80 ms (0%) 31 ms (+51.7% 🔺) 110 ms
viem/accounts 89.01 KB (0%) 1.8 s (0%) 64 ms (+6.62% 🔺) 1.9 s
viem/accounts (tree-shaking) 19.37 KB (0%) 388 ms (0%) 48 ms (+99.75% 🔺) 435 ms
viem/actions 43.25 KB (-0.01% 🔽) 865 ms (-0.01% 🔽) 567 ms (+3.26% 🔺) 1.5 s
viem/actions (tree-shaking) 350 B (0%) 10 ms (0%) 28 ms (-6.98% 🔽) 38 ms
viem/chains 59.92 KB (0%) 1.2 s (0%) 101 ms (+35.29% 🔺) 1.3 s
viem/chains (tree-shaking) 470 B (0%) 10 ms (0%) 29 ms (+74.7% 🔺) 39 ms
viem/chains/utils 7.97 KB (0%) 160 ms (0%) 27 ms (-8.04% 🔽) 187 ms
viem/chains/utils (tree-shaking) 5.24 KB (0%) 105 ms (0%) 39 ms (+80.01% 🔺) 144 ms
viem/ens 43.25 KB (0%) 865 ms (0%) 501 ms (-16.14% 🔽) 1.4 s
viem/ens (tree-shaking) 17.94 KB (0%) 359 ms (0%) 46 ms (+45.68% 🔺) 404 ms

@zeroXbrock zeroXbrock merged commit 107f633 into main Dec 8, 2023
6 of 13 checks passed
@zeroXbrock zeroXbrock deleted the fix-ci branch December 8, 2023 01:55
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.

1 participant