Skip to content

Conversation

@themacexpert
Copy link
Contributor

@themacexpert themacexpert commented Dec 7, 2025

Description

Slims down tutorials section according to docs overhaul guide.

This pull request removes a number of code and documentation files related to Ethereum smart contract tutorials. The deleted files include Solidity contracts, test scripts, deployment scripts, and markdown files showing terminal output and code snippets. The changes suggest a significant cleanup or restructuring of the tutorial content, possibly to replace or update them elsewhere.

Goes with: papermoonio/moonbeam-mkdocs#287

The most important changes are:

Removal of Solidity contracts and test files:

  • Deleted core contract files such as Container.sol, MyToken.sol, and their related test files (Container.t.sol, MyToken.t.sol, ContainerHarness.sol, etc.), as well as deployment and script files (ContainerDeployScript.sol, Container.s.sol). [1] [2] [3] [4] [5] [6] [7]
  • Removed auxiliary test and fuzzing files (Fuzz-test.sol, Fuzz-test2.sol, IsOverflowing.sol, TestAlternateTokenOnMoonbaseFork.sol). [1] [2] [3] [4]

Removal of batch approve & swap example:

  • Deleted the example script swap.js for batch approving and swapping tokens, along with related terminal output markdown files for compile, deploy, and swap steps. [1] [2] [3] [4]

Removal of terminal output documentation:

  • Removed markdown files documenting the output of compiling, deploying, and testing contracts (terminal/compile.md, terminal/deploy.md, terminal/swap.md, terminal/test.md, terminal/test2.md). [1] [2] [3]

These changes collectively remove the code and supporting documentation for several Ethereum contract tutorials and their associated test and deployment workflows.

Checklist

  • Added a label 🏷️ to this PR
  • Ran my changes through Grammarly
  • Added a disclaimer if required
  • If pages were moved, opened a corresponding PR in moonbeam-mkdocs to update redirects

Translations

Does this PR update a page that also exists on the Chinese docs site? See mapping to confirm.

  • Yes
  • No

If Yes, complete the following:

  • Opened a PR on the Chinese docs repo for the corresponding page
  • Updated images, snippets, or variables if they were moved, renamed, or deleted

Link to the corresponding CN docs PR: <INSERT_LINK>

Copilot AI review requested due to automatic review settings December 7, 2025 02:57
@themacexpert themacexpert requested a review from a team as a code owner December 7, 2025 02:57
@themacexpert themacexpert added A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B0 - Needs Review Pull request is ready for review labels Dec 7, 2025
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 pull request removes a significant number of tutorial files as part of a documentation overhaul initiative. The changes streamline the tutorials section by removing entire categories of content related to interoperability, integrations, and some Ethereum API tutorials.

Key Changes:

  • Complete removal of the interoperability tutorials section (XCM and protocol-based tutorials)
  • Removal of integration tutorials (indexers and oracles)
  • Deletion of several Ethereum API tutorials (thirdweb, randomness lottery, ChatGPT, Hardhat workflow, and DApp building)
  • Author byline removal from the Tenderly tutorial

Reviewed changes

Copilot reviewed 156 out of 220 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tutorials/interoperability/*.md Removed all interoperability tutorials including XCM examples, Axelar SDK, Uniswap V2 swap, remote staking, remote batched EVM calls, and cross-chain DAO
tutorials/interoperability/*.index.md Removed index pages for XCM and protocols subsections
tutorials/integrations/*.md Removed integration tutorials for SQD/Subsquid (NFT indexing and local dev node) and Supra Oracles
tutorials/integrations/*.index.md Removed index pages for indexers and oracles subsections
tutorials/eth-api/*.md Removed tutorials for thirdweb, randomness lottery, ChatGPT smart contracts, Hardhat workflow, and DApp building guide
tutorials/eth-api/*.index.md Removed index pages for get-started and precompiles subsections
tutorials/.nav.yml Updated to remove interoperability section reference
tutorials/integrations/.nav.yml Removed all indexer and oracle tutorial references
tutorials/eth-api/using-tenderly.md Removed author byline only

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

eshaben
eshaben previously approved these changes Dec 16, 2025
@eshaben
Copy link
Contributor

eshaben commented Dec 16, 2025

oops @themacexpert merge conflicts

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

Copilot reviewed 156 out of 220 changed files in this pull request and generated no new comments.


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

@themacexpert themacexpert marked this pull request as draft December 17, 2025 00:23
@themacexpert themacexpert marked this pull request as ready for review December 17, 2025 00:23
@eshaben
Copy link
Contributor

eshaben commented Dec 17, 2025

@themacexpert more merge conflicts 😭

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

Copilot reviewed 153 out of 217 changed files in this pull request and generated no new comments.


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

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

Copilot reviewed 153 out of 217 changed files in this pull request and generated no new comments.


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

Copy link
Contributor

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

merge conflicts, at least it's on a deleted page haha should be easy to resolve 😅

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

Copilot reviewed 153 out of 217 changed files in this pull request and generated 2 comments.


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

@eshaben eshaben merged commit 28d2268 into master Dec 22, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A1 - Maintenance Major Pull request contains major updates to an existing page (i.e., adding a new section, reorgs, etc.) B0 - Needs Review Pull request is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants