Skip to content

fix: correct spelling in error messages and documentation #2840

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.

Sign up for GitHub

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

Jump to bottom
Merged
yingjiehe-xyz merged 1 commit into from
Jun 13, 2025
Merged

fix: correct spelling in error messages and documentation #2840

yingjiehe-xyz merged 1 commit into from
Jun 13, 2025

Conversation

@sfc-gh-twhite
Copy link
Contributor

@sfc-gh-twhite sfc-gh-twhite commented Jun 9, 2025

This PR corrects various spelling errors found throughout the codebase in error messages, documentation, and configuration files.

Changes Made

Error Messages & Documentation: Fixed spelling errors in multiple Rust files:

  • extension.rs: Fixed "accomodate" → "accommodate" in context limit error message
  • router_tools.rs: Fixed "Imporant" → "Important" in tool selection instructions
  • system_gpt_4.1.md: Fixed "CRITIAL" → "CRITICAL" in system prompt documentation
  • oauth.rs: Fixed "recieve" → "receive" in OAuth flow comment
  • main.rs (mcp-server): Fixed "agrument" → "argument" in example prompt description

Configuration Files:

  • kubernetes_swagger.json: Fixed "classses" → "classes" in DeviceClassSpec descriptions (2 instances)

@sfc-gh-twhite
Copy link
Contributor Author

sfc-gh-twhite commented Jun 9, 2025

I used https://github.com/codespell-project/codespell to do these checks. I didn't modify a few that were provider-specific or valid code shorthands.

I would be happy to help integrate something like this into CI if that would be helpful or desired. 😄

michaelneale
michaelneale approved these changes Jun 10, 2025
View reviewed changes
Copy link
Collaborator

@michaelneale michaelneale left a comment

Choose a reason for hiding this comment

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

nice, some real bad typos in there. that is how you know a human made them!

@yingjiehe-xyz yingjiehe-xyz merged commit 7e40cd4 into block:main Jun 13, 2025
6 checks passed
zanesq added a commit that referenced this pull request Jun 13, 2025
@zanesq
Merge branch 'main' of github.com:block/goose into zane/disable-updater
48070ea 
* 'main' of github.com:block/goose:
  fix: correct spelling in error messages and documentation (#2840)
  Change updater to use platform agnostic and secure zip library (#2913)
  Docs: Edit recipes on Goose desktop (#2912)

# Conflicts:
#	ui/desktop/src/utils/githubUpdater.ts
chaitanyarahalkar pushed a commit to chaitanyarahalkar/goose that referenced this pull request Jun 16, 2025
@sfc-gh-twhite @chaitanyarahalkar
fix: correct spelling in error messages and documentation (block#2840)
b7178cc
This was referenced Jun 16, 2025
Closed
Open
michaelneale added a commit that referenced this pull request Jun 16, 2025
@michaelneale
Merge branch 'main' into micn/fast-edit-morph-model
bd2a88f 
* main: (51 commits)
  Docs: Fetch MCP doesnt work with Gemini (#2940)
  feat: add Help & Feedback section in App Settings (#2935)
  docs: blog update (#2937)
  docs: fixing blog image (#2936)
  docs: lead/worker tutorial and blog post (#2930)
  chore(deps): bump golang.org/x/net from 0.14.0 to 0.38.0 in /temporal-service (#2836)
  chore(deps): bump google.golang.org/grpc from 1.57.0 to 1.57.1 in /temporal-service (#2834)
  fix updater download text (#2919)
  chore(release): release version 1.0.28 (#2906)
  Enable updater and remove unzipping and installing update text (#2918)
  docs: updates for lead-worker model (#2916)
  fix: correct spelling in error messages and documentation (#2840)
  Change updater to use platform agnostic and secure zip library (#2913)
  Docs: Edit recipes on Goose desktop (#2912)
  Disable updater until we can debug more in release (#2908)
  fix router trait error (#2910)
  fix: Check for stderr error in receive() (#2905)
  Damien/sagemaker tgi (#2888)
  feat: (tool router) llm tool selector (#2866)
  feat: (tool router) adds extension name in vector db & search tool (#2855)
  ...
lifeizhou-ap added a commit that referenced this pull request Jun 16, 2025
@lifeizhou-ap
Merge branch 'main' into lifei/sub-recipe-simple
aca1721 
* main: (23 commits)
  Docs: Fetch MCP doesnt work with Gemini (#2940)
  feat: add Help & Feedback section in App Settings (#2935)
  docs: blog update (#2937)
  docs: fixing blog image (#2936)
  docs: lead/worker tutorial and blog post (#2930)
  chore(deps): bump golang.org/x/net from 0.14.0 to 0.38.0 in /temporal-service (#2836)
  chore(deps): bump google.golang.org/grpc from 1.57.0 to 1.57.1 in /temporal-service (#2834)
  fix updater download text (#2919)
  chore(release): release version 1.0.28 (#2906)
  Enable updater and remove unzipping and installing update text (#2918)
  docs: updates for lead-worker model (#2916)
  fix: correct spelling in error messages and documentation (#2840)
  Change updater to use platform agnostic and secure zip library (#2913)
  Docs: Edit recipes on Goose desktop (#2912)
  Disable updater until we can debug more in release (#2908)
  fix router trait error (#2910)
  fix: Check for stderr error in receive() (#2905)
  Damien/sagemaker tgi (#2888)
  feat: (tool router) llm tool selector (#2866)
  feat: (tool router) adds extension name in vector db & search tool (#2855)
  ...
opdich added a commit to opdich/goose that referenced this pull request Jun 16, 2025
@opdich
Merge remote-tracking branch 'upstream/main' into opopadich/issue-1625
6b9e2ba 
* upstream/main:
  Docs: Fetch MCP doesnt work with Gemini (block#2940)
  feat: add Help & Feedback section in App Settings (block#2935)
  docs: blog update (block#2937)
  docs: fixing blog image (block#2936)
  docs: lead/worker tutorial and blog post (block#2930)
  chore(deps): bump golang.org/x/net from 0.14.0 to 0.38.0 in /temporal-service (block#2836)
  chore(deps): bump google.golang.org/grpc from 1.57.0 to 1.57.1 in /temporal-service (block#2834)
  fix updater download text (block#2919)
  chore(release): release version 1.0.28 (block#2906)
  Enable updater and remove unzipping and installing update text (block#2918)
  docs: updates for lead-worker model (block#2916)
  fix: correct spelling in error messages and documentation (block#2840)
  Change updater to use platform agnostic and secure zip library (block#2913)
  Docs: Edit recipes on Goose desktop (block#2912)
s-soroosh pushed a commit to s-soroosh/goose that referenced this pull request Jul 18, 2025
@sfc-gh-twhite @s-soroosh
fix: correct spelling in error messages and documentation (block#2840)
b079ef1 
Signed-off-by: Soroosh <[email protected]>
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
@sfc-gh-twhite
fix: correct spelling in error messages and documentation (block#2840)
c74d560
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michaelneale michaelneale michaelneale approved these changes

@yingjiehe-xyz yingjiehe-xyz yingjiehe-xyz approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sfc-gh-twhite @michaelneale @yingjiehe-xyz