Skip to content

Fix typos: overriden -> overridden, independantly -> independently#7809

Open
tysoncung wants to merge 1 commit into
microsoft:mainfrom
tysoncung:fix/typo-overridden-updated
Open

Fix typos: overriden -> overridden, independantly -> independently#7809
tysoncung wants to merge 1 commit into
microsoft:mainfrom
tysoncung:fix/typo-overridden-updated

Conversation

@tysoncung

Copy link
Copy Markdown

Replaces PR #7251 (by verreaa) — rebased on current main.

Fixed minor spelling typos across the codebase:

  • overridenoverridden in OpenAI and Ollama config docstrings
  • overridenoverridden in FAQ docs
  • independantlyindependently in semantic router sample README

CC: @verreaa (original author) @anandfresh (original reviewer)

Cherry-pick from PR microsoft#7251 by verreaa.
- overriden -> overridden (Ollama/OpenAI config docs, FAQ)
- independantly -> independently (semantic router README)
@tysoncung

Copy link
Copy Markdown
Author

👋 All CI workflows need approval to run (fork PR). Could a maintainer approve the CI run? CLA and GitGuardian already passed. Thanks! 🙏

@avinashkamat48 avinashkamat48 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This fixes “overriden”, but the FAQ sentence still reads “which will can be overridden”. Since this PR is already touching that line, could you also clean up the surrounding grammar, e.g. “which can be overridden by passing parameters with the same names via extra_grpc_config”?

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.

3 participants