Skip to content

update docs with proxy information#251

Merged
alcuadrado merged 1 commit into
mainfrom
gas-stats-proxy
Apr 16, 2026
Merged

update docs with proxy information#251
alcuadrado merged 1 commit into
mainfrom
gas-stats-proxy

Conversation

@schaable
Copy link
Copy Markdown
Member

@schaable schaable self-assigned this Apr 10, 2026
Copilot AI review requested due to automatic review settings April 10, 2026 15:04
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 10, 2026

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

Project Deployment Actions Updated (UTC)
hardhat-website Ready Ready Preview, Comment Apr 10, 2026 3:05pm

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 gas statistics guide to document proxy-aware gas stats JSON output (as introduced in NomicFoundation/hardhat#8119), so consumers can interpret direct vs proxied call entries correctly.

Changes:

  • Adds proxyChain to the JSON example for direct-call contract entries.
  • Extends the JSON example with a proxied-call contract entry and deployment: null.
  • Adds an “Understanding the JSON structure” section explaining display labels vs programmatic fields.

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

Comment thread src/content/docs/docs/guides/testing/gas-statistics.mdx
Comment thread src/content/docs/docs/guides/testing/gas-statistics.mdx
@schaable schaable requested a review from alcuadrado April 10, 2026 15:11
@alcuadrado alcuadrado added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit b16ff8e Apr 16, 2026
7 checks passed
@alcuadrado alcuadrado deleted the gas-stats-proxy branch April 16, 2026 21:29
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.

3 participants