Skip to content

Conversation

@heijiLee
Copy link
Contributor

Description

Closes: #755

This PR fixes a small typo in the gov precompile error message.
NewMsgCancelProposal takes 2 arguments, but the error message says it requires 1. (ref)
This is a minor fix for readability and consistency.
Also fix testcase :)

Changes

fmt.Sprintf(cmn.ErrInvalidNumberOfArgs, 1, 0)fmt.Sprintf(cmn.ErrInvalidNumberOfArgs, 2, 0)


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

@heijiLee heijiLee force-pushed the chore/fix-error-typo branch from 1082f63 to 309843c Compare October 28, 2025 07:10
@heijiLee
Copy link
Contributor Author

sqaush & commit because of verified commit signature

@almk-dev almk-dev enabled auto-merge October 28, 2025 15:36
@almk-dev almk-dev added this pull request to the merge queue Oct 28, 2025
Merged via the queue into cosmos:main with commit 0d55db8 Oct 28, 2025
20 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.

typo in error message in gov precompile

3 participants