Skip to content

Conversation

themacexpert
Copy link
Contributor

Description

This pull request updates the runtime upgrade history tables across several documentation and reference files to reflect the release of runtime version 3900 on the Moonriver network. Previously, the table indicated that this upgrade had not been released on Moonriver; the change adds the correct block reference for Moonriver, ensuring consistency and accuracy in the documentation.

Updates to runtime upgrade tables:

  • Added the Moonriver block reference (13423617) for runtime upgrade 3900 in the following files:
    • builders/build/runtime-upgrades.md
    • llms-files/llms-dev-environments.txt
    • llms-files/llms-ethereum-toolkit.txt
    • llms-files/llms-gmp-providers.txt
    • llms-files/llms-governance.txt
    • llms-files/llms-indexers-and-queries.txt
    • llms-files/llms-integrations.txt
    • llms-files/llms-json-rpc-apis.txt
    • llms-files/llms-libraries-and-sdks.txt
    • llms-files/llms-node-operators-and-collators.txt
    • llms-files/llms-oracle-nodes.txt

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>

@themacexpert themacexpert requested a review from a team as a code owner October 6, 2025 17:19
@Copilot Copilot AI review requested due to automatic review settings October 6, 2025 17:19
@themacexpert themacexpert added A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) B0 - Needs Review Pull request is ready for review labels Oct 6, 2025
@themacexpert themacexpert requested a review from eshaben October 6, 2025 17:19
Copy link
Contributor

@Copilot 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 updates runtime version references across the Moonbeam documentation ecosystem from v0.47.0 to v0.47.1, and adds the missing Moonriver block reference (13423617) for runtime upgrade 3900. The update ensures that all documentation reflects the current runtime version and provides complete deployment information for the 3900 upgrade across all networks.

Key changes:

  • Updated Docker image tags and binary version references from v0.47.0 to v0.47.1
  • Added Moonriver block reference for runtime upgrade 3900 in all documentation tables
  • Updated Moonriver spec version from runtime-3800 to runtime-3900

Reviewed Changes

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

Show a summary per file
File Description
variables.yml Updated version tags and SHA checksums for v0.47.1 across all network configurations
llms-full.txt Updated runtime upgrade table with Moonriver block reference and version strings throughout
builders/build/runtime-upgrades.md Added missing Moonriver block reference for runtime 3900
Multiple llms-files/*.txt Propagated version updates and runtime table changes across all LLM documentation files
Multiple .snippets files Updated Docker commands and terminal output examples to reflect v0.47.1

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

github-actions bot commented Oct 6, 2025

Style Guide Violations Summary

.snippets/code/builders/ethereum/json-rpc/debug-trace/terminal/start-up-logs.md (Line 14)

  • Issue:
    Emoji used in terminal output violates 'Do not use emojis' rule in Symbols section

  • Correction:

<br> 2025-07-10 09:04:26 version 0.47.1-d7df89e7161

.snippets/code/builders/get-started/networks/moonbeam-dev/terminal/docker-pull.md (Line 14)

  • Issue:
    Terminal output should end with a blank command prompt to indicate command completion per style guide

  • Correction:

<span data-ty="input"><span class="file-path"></span></span>

.snippets/code/builders/get-started/networks/moonbeam-dev/terminal/docker-run.md (Line 9)

  • Issue:
    Use of emoji violates style guide (Symbols section: 'Do not use emojis')

  • Correction:

<br> 2025-07-10 09:04:26 version 0.47.1-d7df89e7161

.snippets/code/builders/get-started/networks/moonbeam-dev/terminal/run-binary.md (Line 5)

  • Issue:
    Emoji usage violates style guide rule 'Do not use emojis' under Symbols section

  • Correction:

<br> 2025-07-10 09:04:26 version 0.47.1-d7df89e7161

.snippets/code/node-operators/networks/run-a-node/docker/terminal/start.md (Line 2)

  • Issue:
    Contains space in path '/d ata' and incorrect user ID syntax; double dash '----name' should be single dash '--name'

  • Correction:

<span data-ty="input"><span class="file-path"></span>docker run --network="host" -v "/var/lib/alphanet-data:/data" -u $(id -u ${USER}):$(id -g ${USER}) moonbeam-foundation/moonbeam:v0.47.1 --base-path=/data --chain alphanet --name="TestNode" --state-pruning archive --trie-cache-size 1073741824 --db-cache 8000 --name="TestNode (Embedded Relay)"</span>

.snippets/code/node-operators/networks/run-a-node/docker/terminal/start.md (Line 4)

  • Issue:
    Contains emoji '✌️' which violates style guide rule against using emojis

  • Correction:

<span data-ty>2025-07-10 09:04:26 version 0.47.1-d7df89e7161 </span>

.snippets/code/tutorials/integrations/local-subsquid/terminal/start-node.md (Line 6)

  • Issue:
    Emoji used in terminal output violates 'Do not use emojis' rule in Symbols section

  • Correction:

<span data-ty>2025-07-10 09:04:26 version 0.47.1-d7df89e7161</span>

builders/build/runtime-upgrades.md (Line 100)

  • Issue:
    Inconsistent spacing in table cells affecting visual alignment

  • Correction:

|                                 [3900](https://forum.moonbeam.network/t/runtime-rt3900-schedule/2262){target=_blank}                                  |                                   -                                    | [13423617](https://moonriver.subscan.io/block/13423617){target=_blank} | [13744972](https://moonbase.subscan.io/block/13744972){target=_blank} |

variables.yml (Line 19)

  • Issue:
    Use of 'this' in comment violates link text guidelines (should be descriptive rather than generic)

  • Correction:

parachain_release_tag: v0.47.1 # must be in 'vX.Y.Z' format for links to work

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.

Just want to double-check on these things:

  • I see the variables.yml file updated the spec_version, does runtime upgrades by block need to be updated? Or was that done in another PR?
  • As a result of the spec_version change, did you double-check the links to see if they still point to the same lines of code?

@themacexpert
Copy link
Contributor Author

Just want to double-check on these things:

  • I see the variables.yml file updated the spec_version, does runtime upgrades by block need to be updated? Or was that done in another PR?
  • As a result of the spec_version change, did you double-check the links to see if they still point to the same lines of code?

Good point on #2, fixed!
Runtime upgrades was updated, might have been obscured in the sea of LLMs files

Copy link
Contributor

@Copilot 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 30 out of 30 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or 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.

Beautiful. Thanks!

@eshaben eshaben added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Oct 8, 2025
@eshaben
Copy link
Contributor

eshaben commented Oct 8, 2025

@themacexpert failing link check. I think this was already fixed, maybe just need to pull in latest changes from master?

@eshaben eshaben removed the B1 - Ready to Merge Pull request is ready to be merged label Oct 8, 2025
@eshaben eshaben self-assigned this Oct 8, 2025
@themacexpert
Copy link
Contributor Author

Wooo

@themacexpert themacexpert added the B1 - Ready to Merge Pull request is ready to be merged label Oct 8, 2025
@eshaben
Copy link
Contributor

eshaben commented Oct 10, 2025

Thanks!

@eshaben eshaben merged commit cdcaaa3 into master Oct 10, 2025
3 checks passed
@eshaben eshaben deleted the themacexpert/3900mr branch October 10, 2025 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A3 - Quick Fixes Pull request contains small fixes (i.e., fixing a typo, removing old images, etc.) B1 - Ready to Merge Pull request is ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants