Skip to content

Conversation

@filipdjokic
Copy link
Contributor

No description provided.

@ankurdotb
Copy link
Contributor

Copy link

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 removes (comments out) the Cosmos SDK mempool declaration from the app.toml configuration file during node installation and configuration. The change adds a new method remove_mempool_section() that is called as part of the node configuration process.

  • Introduces a new method to comment out the [mempool] section in app.toml
  • Integrates this removal into the standard node configuration workflow

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

Copy link
Member

@Eengineer1 Eengineer1 left a comment

Choose a reason for hiding this comment

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

Minor improvements to be made; looks very good in terms of functionality.

@filipdjokic filipdjokic merged commit 394b086 into develop Dec 15, 2025
31 of 34 checks passed
@filipdjokic filipdjokic deleted the remove-mempool-declaration-from-config branch December 15, 2025 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants