Skip to content

docs: update gas statistics page for aggregated deployment stats#232

Merged
kanej merged 1 commit intomainfrom
deployment-gas-stats
Mar 31, 2026
Merged

docs: update gas statistics page for aggregated deployment stats#232
kanej merged 1 commit intomainfrom
deployment-gas-stats

Conversation

@schaable
Copy link
Copy Markdown
Member

@schaable schaable commented Mar 7, 2026

@schaable schaable self-assigned this Mar 7, 2026
Copilot AI review requested due to automatic review settings March 7, 2026 00:12
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hardhat-website Ready Ready Preview, Comment Mar 7, 2026 0:12am

Request Review

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Hardhat gas statistics documentation to reflect the new aggregated deployment statistics format (related to hardhat#8038).

Changes:

  • Updates the sample output tables to show deployment stats as aggregated Min/Average/Median/Max with #deployments.
  • Updates the “Understanding the Gas Statistics Table” section to describe deployments alongside function calls.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +68 to 72
- **count**: Number of times the function was called or the contract was deployed
- **min**: Minimum gas consumed in a single call
- **max**: Maximum gas consumed in a single call
- **avg**: Average gas consumed across all calls
- **median**: Median gas consumed across all calls
Copy link

Copilot AI Mar 7, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The metric definitions still describe only function calls (e.g., “in a single call” / “across all calls”), but this section now says the table includes deployments too. Please update these bullets to describe both (e.g., call or deployment) and align the naming with the table headers (#calls vs #deployments) so readers aren’t confused about what “count” refers to for deployments.

Suggested change
- **count**: Number of times the function was called or the contract was deployed
- **min**: Minimum gas consumed in a single call
- **max**: Maximum gas consumed in a single call
- **avg**: Average gas consumed across all calls
- **median**: Median gas consumed across all calls
- **count**: For functions, the number of calls (`#calls`); for deployments, the number of times the contract was deployed (`#deployments`)
- **min**: Minimum gas consumed in a single call or deployment
- **max**: Maximum gas consumed in a single call or deployment
- **avg**: Average gas consumed across all calls or deployments
- **median**: Median gas consumed across all calls or deployments

Copilot uses AI. Check for mistakes.
@kanej kanej removed their request for review March 12, 2026 15:18
@kanej kanej added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit 1af4929 Mar 31, 2026
7 checks passed
@kanej kanej deleted the deployment-gas-stats branch March 31, 2026 14:22
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.

4 participants