Skip to content

Conversation

@fvictorio
Copy link
Contributor

The simulated networks explanation was actually a mix of explanation and list of features. I think the latter is appropiate for a reference, even if it's not an API-like reference.

This PR moves most of the content of the explanation to this new reference, plus two other minor changes (see individual commits).

Copilot AI review requested due to automatic review settings November 19, 2025 15:29
@vercel
Copy link

vercel bot commented Nov 19, 2025

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

Project Deployment Preview Comments Updated (UTC)
hardhat-website Ready Ready Preview Comment Nov 19, 2025 3:29pm

Copy link
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

This PR restructures the EDR simulated networks documentation by separating conceptual explanations from reference material. The main purpose is to move the detailed feature list and configuration examples from the explanation document to a new dedicated reference page, making the documentation more navigable and appropriately organized.

Key changes:

  • Creates a new reference document for simulated networks with comprehensive feature details
  • Streamlines the explanation document to focus on conceptual overview
  • Improves internal linking between the two documents

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/content/docs/docs/reference/edr-simulated-networks.mdx New reference document containing detailed feature listings, configuration examples, and behavior specifications previously in the explanation
src/content/docs/docs/explanations/edr-simulated-networks.mdx Shortened to focus on conceptual overview, with most technical details moved to the reference document

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

@fvictorio fvictorio mentioned this pull request Nov 19, 2025
17 tasks
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.

2 participants