Skip to content

fix: correct spelling in comments and tests#9702

Open
HusseinAdeiza wants to merge 1 commit into
osmosis-labs:mainfrom
HusseinAdeiza:fix/comment-typos
Open

fix: correct spelling in comments and tests#9702
HusseinAdeiza wants to merge 1 commit into
osmosis-labs:mainfrom
HusseinAdeiza:fix/comment-typos

Conversation

@HusseinAdeiza

Copy link
Copy Markdown

Description

Fixed spelling errors in comments and test files.

⚠️ Note: Comment and test changes only. No functional code modified.

Changes

  • Fix 'cant' → 'can't'
  • Fix 'dont' → 'don't'

Impact

  • Comment clarity improvement only
  • No functional changes

- Fix 'dont' → 'don't'
- Fix 'cant' → 'can't'

⚠️ Note: Comment and test changes only. No functional code modified.
@github-actions github-actions Bot added the C:simulator Edits simulator or simulations label May 23, 2026
@coderabbitai

coderabbitai Bot commented May 23, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 7d395c1b-aeca-47c0-bc08-edee8733a9bb

📥 Commits

Reviewing files that changed from the base of the PR and between 2df8d8e and a155828.

📒 Files selected for processing (4)
  • cmd/osmosisd/cmd/init.go
  • simulation/executor/internal/executortypes/simmanager.go
  • x/ibc-rate-limit/contracts/rate-limiter/src/integration_tests.rs
  • x/valset-pref/validator_set.go

Walkthrough

This pull request corrects comment punctuation and grammar across four modules: the initialization command, simulation executor, rate-limiter contract integration tests, and validator set preference logic. All changes are comment-only with no executable code modifications.

Changes

Comment cleanup across modules

Layer / File(s) Summary
Comment punctuation and grammar corrections
cmd/osmosisd/cmd/init.go, simulation/executor/internal/executortypes/simmanager.go, x/ibc-rate-limit/contracts/rate-limiter/src/integration_tests.rs, x/valset-pref/validator_set.go
Comments are corrected for contractions ("don't" vs "dont", "can't" vs "cant") and wording clarity in initialization log messaging, simulation executor loop documentation, governance-permission test assertions, and validator set delegation rounding behavior explanation.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description covers the purpose and impact, but lacks required sections like 'Testing and Verifying' and 'Documentation and Release Note' from the template. Add 'Testing and Verifying' section confirming this is a trivial code cleanup, and complete 'Documentation and Release Note' section with appropriate checkboxes.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: correcting spelling in comments and tests.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.12.2)

level=error msg="[linters_context] typechecking error: pattern ./...: directory prefix . does not contain main module or its selected dependencies"


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had any recent activity. It will be closed if no further activity occurs. Thank you!

@github-actions github-actions Bot added the Stale label Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C:simulator Edits simulator or simulations Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant