Skip to content

Remove bstm gas meter docs as unnecessary#260

Merged
Eric-Warehime merged 1 commit intomainfrom
eric/update-bstm-docs
Mar 23, 2026
Merged

Remove bstm gas meter docs as unnecessary#260
Eric-Warehime merged 1 commit intomainfrom
eric/update-bstm-docs

Conversation

@Eric-Warehime
Copy link
Contributor

Updates the docs to reflect changes in cosmos/cosmos-sdk#26138

@greptile-apps
Copy link

greptile-apps bot commented Mar 19, 2026

Greptile Summary

This PR removes the "Disable the Block Gas Meter" sub-section from the Block-STM parallel execution documentation (sdk/next/experimental/blockstm.mdx), reflecting the upstream change in cosmos/cosmos-sdk#26138 that makes SetDisableBlockGasMeter unnecessary (or no longer available).

  • The removed section described calling bApp.SetDisableBlockGasMeter(true) as a prerequisite for parallel execution; this is no longer required.
  • The surrounding documentation (Virtual Fee Collection, Object Store setup, Custom Modules) remains intact and reads coherently without the deleted section.
  • No internal links or cross-references within the file point to the removed heading.
  • References to blockGasMeter found elsewhere in the repo are in versioned SDK docs (v0.47, v0.50, v0.53) that document the general gas-meter feature and are unrelated to this Block-STM-specific removal.

Confidence Score: 5/5

  • This is a safe, minimal docs-only change with no code or logic impact.
  • The PR removes a single, self-contained documentation section. The change is well-motivated (upstream SDK API removal), the file reads correctly after the deletion, and no other files in the repository reference this specific heading.
  • No files require special attention.

Important Files Changed

Filename Overview
sdk/next/experimental/blockstm.mdx Removes the "Disable the Block Gas Meter" section (8 lines) from the Block-STM integration guide; the surrounding content remains coherent and no internal links are broken.

Last reviewed commit: "Remove bstm gas mete..."

Copy link
Contributor

@evanorti evanorti left a comment

Choose a reason for hiding this comment

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

LGTM!

Merge away

@Eric-Warehime Eric-Warehime merged commit 46dc5bb into main Mar 23, 2026
4 checks passed
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