Skip to content

docs: clarify that solidity --snapshot-check is read-only - #278

Merged
schaable merged 3 commits into
mainfrom
fix/read-only-snapshot-check
Jul 2, 2026
Merged

schaable merged 3 commits into
mainfrom
fix/read-only-snapshot-check

Conversation

@schaable

@schaable schaable commented Jun 13, 2026 •

Copy link
Copy Markdown
Member

Adds a note to the gas snapshots guide explaining that hardhat test solidity --snapshot-check only reads the snapshot files and never modifies them.

Related to: NomicFoundation/hardhat#8391.

@schaable
schaable requested a review from Copilot June 13, 2026 20:28
@schaable schaable self-assigned this Jun 13, 2026
@vercel

vercel Bot commented Jun 13, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
hardhat-website Ready Ready Preview, Comment Jun 22, 2026 2:26pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the gas snapshots documentation to clarify that --snapshot-check is a read-only operation and won’t create or modify snapshot files, helping users understand when to use --snapshot vs --snapshot-check.

Changes:

  • Add a :::note explaining that --snapshot-check only reads snapshot files and what happens when no snapshots exist yet.

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

Comment thread src/content/docs/docs/guides/testing/gas-snapshots.mdx Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@schaable
schaable added this pull request to the merge queue Jul 2, 2026
Merged via the queue into main with commit 4249c91 Jul 2, 2026
3 checks passed
@schaable
schaable deleted the fix/read-only-snapshot-check branch July 2, 2026 15:59

This branch was successfully deployed

1 active deployment
Preview — 444c5e72 Deployed Jun 22, 2026 by vercel[bot]
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