-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Document Formatting #7222
Document Formatting #7222
Conversation
Bundle StatsHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger No assets were bigger Smaller No assets were smaller Unchanged
|
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.
Benchmark
Benchmark suite | Current: 608d0b9 | Previous: b86d8ca | Ratio |
---|---|---|---|
processingTx |
22506 ops/sec (±7.53% ) |
22440 ops/sec (±7.06% ) |
1.00 |
processingContractDeploy |
40209 ops/sec (±7.30% ) |
38564 ops/sec (±7.90% ) |
0.96 |
processingContractMethodSend |
16251 ops/sec (±9.26% ) |
16696 ops/sec (±7.81% ) |
1.03 |
processingContractMethodCall |
28936 ops/sec (±5.93% ) |
27650 ops/sec (±7.82% ) |
0.96 |
abiEncode |
44377 ops/sec (±7.88% ) |
44651 ops/sec (±6.88% ) |
1.01 |
abiDecode |
31746 ops/sec (±6.86% ) |
30424 ops/sec (±7.61% ) |
0.96 |
sign |
1575 ops/sec (±0.58% ) |
1574 ops/sec (±0.58% ) |
1.00 |
verify |
362 ops/sec (±2.57% ) |
365 ops/sec (±2.65% ) |
1.01 |
This comment was automatically generated by workflow using github-action-benchmark.
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.
@danforbes In this doc PR, I'll suggest to create formatting guide using functions in packages like web3.eth.getBlockNumber(..), ...
@jdevcs can you clarify your suggestion? The existing guide uses |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 4.x #7222 +/- ##
=======================================
Coverage 94.44% 94.44%
=======================================
Files 216 216
Lines 8454 8454
Branches 2335 2335
=======================================
Hits 7984 7984
Misses 470 470
Flags with carried forward coverage won't be shown. Click here to find out more. |
what do you think for adding one example for each of sub sections specially for formatting numbers using |
@jdevcs is this what you had in mind? https://f3916c2c.web3-js-docs.pages.dev/guides/getting_started/return-formats |
yes, looks very good, |
Document formatting - closes #6743