Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: multiple typos of different importance #2571

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

crStiv
Copy link

@crStiv crStiv commented Dec 19, 2024

Description

This pull request addresses multiple typographical errors across different files in the project. The corrections include fixing spelling errors and redundant phrases in comments and documentation. These changes improve the readability and clarity of the codebase.

Changes Introduced

  1. .github/workflows/test.yml:

    • Fixed the typo in "Pull Requeset" to "Pull Request."
  2. app/upgrades/mainnet/v1/upgrades.go:

    • Corrected the redundant phrase "account address address" to "account address."
  3. app/upgrades/testnet/v4/upgrades.go:

    • Replaced "atleast" with "at least."
  4. app/upgrades/testnet/v5/upgrades.go:

    • Fixed the typo "currect" to "current."

Author Checklist

  • Included the correct type prefix in the PR title.
  • Targeted the correct branch as per the PR Targeting guidelines.
  • Confirmed the changes are non-breaking and do not require additional tests.
  • Updated the necessary documentation where applicable.
  • Verified that all CI checks pass successfully.

Reviewers Checklist

  • Verified the correct type prefix in the PR title.
  • Confirmed all author checklist items have been addressed.
  • Reviewed and confirmed the accuracy of the changes.
  • Verified the changes improve code clarity and maintainability.
  • Confirmed all CI checks have passed.

Closes: #XXXX

Allow edits by maintainers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant