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: typos in documentation files #5397

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

Conversation

kilavvy
Copy link

@kilavvy kilavvy commented Feb 26, 2025

Corrected "upadte" → "update"
Corrected "unqiue" → "unique"
Corrected "menmonic" → "mnemonic"
Corrected "curent" → "current"
Corrected "verson" → "version"

@kilavvy kilavvy requested review from a team as code owners February 26, 2025 15:02
@kilavvy
Copy link
Author

kilavvy commented Mar 2, 2025

@amitabh94 @infiniteflower Please check this PR whenever you have a free moment.

@mcmire
Copy link
Contributor

mcmire commented Mar 12, 2025

@kilavvy Hmm, the PR description and title seems misleading here. It sounds like we are making changes to constructor options for various controllers, but that is not the case. It appears that this PR fixes some typos in JSDoc or messaging. Can you update the PR title and description to match?

@kilavvy kilavvy changed the title Add configuration updates for various controllers fix: typos in documentation files Mar 12, 2025
@kilavvy
Copy link
Author

kilavvy commented Mar 12, 2025

@kilavvy Hmm, the PR description and title seems misleading here. It sounds like we are making changes to constructor options for various controllers, but that is not the case. It appears that this PR fixes some typos in JSDoc or messaging. Can you update the PR title and description to match?

updated

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