-
Notifications
You must be signed in to change notification settings - Fork 723
Fix spelling and grammatical errors across repository #11455
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
base: master
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-authored-by: Byron <63622+Byron@users.noreply.github.com>
402acb9 to
0bf1211
Compare
0bf1211 to
3280daa
Compare
3280daa to
f80b63a
Compare
|
This PR was motivated by a user-facing spelling mistake that we could certainly have avoided if we would use Copilot for reviews. Simple things it will definitely catch, and often better than humans when it comes to typos. In any case, I reviewed every change it proposed and fixed up the rest, so probably there is nothing user-facing left now. There was also a specification of It's notable that there were a couple of 'dumb' renames that I think it, as "AI", should have figured out as intentional. That's a bit of a bummer. Finally, there wan't anything user facing except for error strings, and our release manifest for flatpaks. I have a feeling that this playwright test might be flaky, as the message the editor recorded clearly was missing a portion ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR systematically corrects 383 spelling and grammatical errors across 184 files in the repository, improving code professionalism and reducing potential confusion during code review. The changes span documentation, comments, variable names, error messages, and UI text across both the Rust backend and TypeScript/Svelte frontend.
Key Changes:
- Fixed common spelling errors (accommodate, uncommitted, assignments, dependencies, etc.)
- Corrected grammar issues (doesn't, can't, that's)
- Renamed variables with typos (
filenName→filename,walkininSvg→walkingSvg) - Updated technical terminology (committed, corresponding, accessibility)
Reviewed changes
Copilot reviewed 186 out of 192 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| packages/ui/src/stories/docs/colors.mdx | Fixed spelling: "useage" → "usage", "breaked" → "broken", "preffered" → "preferred" |
| packages/ui/src/lib/utils/diffParsing.ts | Corrected parameter name: filenName → filename (used 3 times) |
| packages/ui/src/lib/richText/RichTextEditor.svelte | Fixed "programatically" → "programmatically" |
| packages/ui/src/lib/components/file/typeMap.ts | Fixed filename mapping: "severless" → "serverless" |
| packages/ui/src/lib/components/file/fileIcons.ts | Fixed icon key: "severless" → "serverless" |
| packages/shared/src/lib/storeUtils.ts | Corrected "vise versa" → "vice versa", "garuntee" → "guarantee" |
| packages/shared/src/lib/redux/store.svelte.ts | Fixed comment: "servies" → "services" |
| packages/shared/src/lib/polling.ts | Fixed "recieved" → "received" |
| packages/shared/src/lib/patchEvents/patchEventsService.ts | Fixed "propogated" → "propagated" |
| packages/shared/src/lib/network/loadable.ts | Fixed "propperly" → "properly" |
| packages/shared/src/lib/interest/interestStore.ts | Fixed "notifer" → "notifier" (2 instances) |
| packages/no-relative-imports/src/paths.ts | Fixed "cooresponding" → "corresponding" |
| packages/no-relative-imports/src/noRelativeImportPaths.test.ts | Fixed "Overriden" → "Overridden" |
| packages/core/src/generated/workspace/refInfo/index.ts | Fixed "traget" → "target" |
| e2e/playwright/tests/upstreamIntegration.spec.ts | Fixed "staus" → "status" |
| crates/gitbutler-workspace/src/branch_trees.rs | Renamed functions: update_uncommited_changes → update_uncommitted_changes, fixed "comptues" → "computes" |
| crates/gitbutler-tauri/src/projects.rs | Fixed "haveing" → "having" |
| crates/gitbutler-tauri/src/main.rs | Updated function call: get_uncommited_files → get_uncommitted_files |
| crates/gitbutler-tauri/src/logs.rs | Fixed "bascially" → "basically" |
| crates/gitbutler-tauri/com.gitbutler.gitbutler.metainfo.xml | Fixed grammar: "cant" → "can't" |
| crates/gitbutler-sync/src/stack_upload.rs | Fixed "immediatly" → "immediately", "dosn't" → "doesn't", "preperation" → "preparation" |
| crates/gitbutler-stack/src/state.rs | Fixed "cant" → "can't" |
| crates/gitbutler-stack/src/stack_branch.rs | Fixed "peformed" → "performed", "refernce" → "reference" |
| crates/gitbutler-stack/src/stack.rs | Fixed "convinience" → "convenience", "performace" → "performance", "refering" → "referring", "differente" → "different", "igrnore" → "ignore" |
| crates/gitbutler-stack/src/lib.rs | Fixed "cant" → "can't" |
| crates/gitbutler-repo/src/repository_ext.rs | Fixed "uncommited" → "uncommitted" |
| crates/gitbutler-project/src/project.rs | Fixed "nexted" → "nested" |
| crates/gitbutler-oplog/src/reflog.rs | Fixed "caues" → "causes" |
| crates/gitbutler-hunk-dependency/src/workspace.rs | Fixed "muiltiple" → "multiple" |
| crates/gitbutler-hunk-dependency/src/path.rs | Fixed "Everytime" → "Every time" |
| crates/gitbutler-hunk-dependency/README.md | Fixed "uncommited" → "uncommitted" (3 instances), "rougly" → "roughly", "exsting" → "existing", "everythin" → "everything", "multple" → "multiple" |
| crates/gitbutler-git/src/executor/mod.rs | Fixed "implementor" → "implementer" (2 instances) |
| crates/gitbutler-filemonitor/vendor/debouncer/tests/fixtures/add_create_dir_event_twice.hjson | Fixed "trough" → "through" |
| crates/gitbutler-filemonitor/vendor/debouncer/src/lib.rs | Fixed "beause" → "because" |
| crates/gitbutler-edit-mode/src/lib.rs | Updated import: update_uncommited_changes_with_tree → update_uncommitted_changes_with_tree |
| crates/gitbutler-cli/src/args.rs | Fixed "brach" → "branch", "direcory" → "directory" |
| crates/gitbutler-branch-actions/tests/virtual_branches/*.rs | Fixed "brnach" → "branch", "seconf" → "second", "verison" → "version", "uncommited" → "uncommitted" |
| crates/gitbutler-branch-actions/tests/reorder.rs | Fixed "Reorded" → "Reordered" |
| crates/gitbutler-branch-actions/tests/fixtures/for-details.sh | Fixed "virutal" → "virtual" |
| crates/gitbutler-branch-actions/tests/dependencies.rs | Fixed "uncommited" → "uncommitted" (4 instances) |
| crates/gitbutler-branch-actions/src/*.rs | Fixed "detatched" → "detached", "mergable" → "mergeable", "preform" → "perform", "cant" → "can't", "caculation" → "calculation", and updated all function calls to use update_uncommitted_changes |
| crates/but/src/tui/select_branch/mod.rs | Fixed "Orderd" → "Ordered" |
| crates/but/src/legacy/id/mod.rs | Fixed "re-use" → "reuse" |
| crates/but/src/command/legacy/status/json.rs | Fixed "Wheter" → "Whether" |
| crates/but/src/command/legacy/mark.rs | Fixed "Cant" → "Cannot" |
| crates/but/src/command/legacy/forge/review.rs | Fixed "targetting" → "targeting" |
| crates/but/src/command/legacy/absorb.rs | Fixed "peformed" → "performed" |
| crates/but/src/args/mod.rs | Fixed "re-used" → "reused", "peformed" → "performed" (2 instances) |
| crates/but-worktrees/tests/worktree/integrate.rs | Fixed "origional" → "original" (2 instances) |
| crates/but-worktrees/src/integrate.rs | Updated function call: update_uncommited_changes → update_uncommitted_changes |
| crates/but-worktrees/src/git.rs | Fixed "parital" → "partial" |
| crates/but-workspace/tests/workspace/tree_manipulation/file.rs | Fixed "ot" → "to" |
| crates/but-workspace/tests/workspace/ref_info/with_workspace_commit/mod.rs | Renamed test functions and variables: dependant → dependent (extensive changes across multiple test functions) |
| crates/but-workspace/tests/workspace/ref_info/with_workspace_commit/legacy.rs | Fixed "dependant" → "dependent" (7 instances) |
| crates/but-workspace/tests/workspace/ref_info/with_workspace_commit/journey/exhaustive_with_squash_merges.rs | Fixed "re-used" → "reused" (2 instances), "conflcit" → "conflict" |
| crates/but-workspace/tests/workspace/commit_engine/refs_update.rs | Fixed "commmit" → "commit" |
| crates/but-workspace/tests/workspace/branch/create_reference.rs | Fixed "completly" → "completely" (2 instances) |
| crates/but-workspace/tests/fixtures/scenario/*.sh | Fixed "depdendent" → "dependent", "re-used" → "reused", "conflcit" → "conflict" |
| crates/but-workspace/src/ui/ref_info.rs | Fixed "traget" → "target" |
| crates/but-workspace/src/tree_manipulation/discard_worktree_changes.rs | Fixed "wether" → "whether" |
| crates/but-workspace/src/legacy/tree_manipulation/utils.rs | Fixed "coorespond" → "correspond", "completly" → "completely" |
| crates/but-workspace/src/legacy/tree_manipulation/*.rs | Fixed "possiblity" → "possibility", "destionation" → "destination", "update_uncommited_changes" → "update_uncommitted_changes" |
| crates/but-workspace/src/legacy/stacks.rs | Fixed "shoudl" → "should", "thats" → "that's", "beloning" → "belonging", "matchies" → "matches", "craete" → "create" |
| crates/but-workspace/src/legacy/branch_details.rs | Fixed "commiter" → "committer" (6 instances) |
| crates/but-workspace/src/commit.rs | Fixed "commiter" → "committer" |
| crates/but-workspace/src/changeset.rs | Fixed "re-use" → "reuse" |
| crates/but-workspace/src/branch_details.rs | Fixed "commiter" → "committer" (6 instances) |
| crates/but-workspace/src/branch/apply.rs | Fixed "branc" → "branch", "retunr" → "return" |
| crates/but-workspace/bindings/workspace/refInfo/index.ts | Fixed "traget" → "target" |
| crates/but-tools/src/workspace.rs | Fixed "excede" → "exceed" (2 instances), "accomodate" → "accommodate", "existin" → "existing" (used incorrectly as "existingg" 6 times), "shuold" → "should", "assingments" → "assignments", "differente" → "different" |
| crates/but-status/src/lib.rs | Fixed "uncommited" → "uncommitted" |
| crates/but-server/src/lib.rs | Updated command name: "get_uncommited_files" → "get_uncommitted_files" |
| crates/but-rules/src/lib.rs | Fixed "determinded" → "determined", "Ammend" → "Amend" |
| crates/but-rebase/src/merge.rs | Fixed "re-used" → "reused" |
| crates/but-rebase/src/commit.rs | Fixed "commiter" → "committer" (2 instances) |
| crates/but-rebase/src/cherry_pick.rs | Fixed "epty" → "empty" |
| crates/but-hunk-dependency/src/ranges/*.rs | Fixed "muiltiple" → "multiple", "Everytime" → "Every time", "ownes" → "owns" |
| crates/but-hunk-dependency/src/input.rs | Fixed "substracting" → "subtracting" |
| crates/but-graph/tests/graph/init/with_workspace.rs | Fixed "re-use" → "reuse", "konw" → "know", "dependant" → "dependent" (15 instances) |
| crates/but-graph/tests/fixtures/scenarios.sh | Fixed "dependant" → "dependent" (4 instances) |
| crates/but-graph/src/projection/workspace.rs | Fixed "re-using" → "reusing" |
| crates/but-graph/src/lib.rs | Fixed "hightlights" → "highlights" |
| crates/but-graph/src/init/post.rs | Fixed "re-used" → "reused" |
| crates/but-error/src/lib.rs | Fixed "meessage" → "message" (2 instances) |
| crates/but-debugging/src/lib.rs | Fixed "infastructure" → "infrastructure" (3 instances) |
| crates/but-cursor/src/lib.rs | Fixed "avaialbe" → "available", "elegibility" → "eligibility" (2 instances) |
| crates/but-core/src/worktree/checkout/function.rs | Fixed "uncommited" → "uncommitted" |
| crates/but-core/src/settings.rs | Fixed "uptodate" → "up-to-date" |
| crates/but-core/src/lib.rs | Fixed "re-use" → "reuse" |
| crates/but-core/src/diff/worktree.rs | Fixed "commiting" → "committing" |
| crates/but-core/src/branch.rs | Fixed "re-use" → "reuse" |
| crates/but-claude/src/rules.rs | Fixed "Thes" → "These" |
| crates/but-claude/src/prompt_templates.rs | Fixed "precidence" → "precedence" |
| crates/but-claude/src/permissions/patterns.rs | Fixed "entirly" → "entirely" (2 instances) |
| crates/but-claude/src/lib.rs | Fixed "requst" → "request" |
| crates/but-claude/src/legacy.rs | Fixed "statuts" → "status" |
| crates/but-claude/src/hooks/mod.rs | Fixed "avaialbe" → "available", "elegibility" → "eligibility" (2 instances) |
| crates/but-claude/src/compact.rs | Fixed "ect" → "etc", "Cant" → "Cannot", "implmentation" → "implementation" |
| crates/but-claude/src/bridge.rs | Fixed "immediatly" → "immediately", "cooresponding" → "corresponding" |
| crates/but-cherry-apply/src/lib.rs | Fixed "hopfully" → "hopefully", "strech" → "stretch", updated function call |
| crates/but-bot/src/state.rs | Fixed "assingments" → "assignments", "defintion" → "definition" |
| crates/but-bot/src/butbot.rs | Fixed "branche" → "branch", "assingments" → "assignments" |
| crates/but-api/src/legacy/workspace.rs | Fixed "alos" → "also" |
| crates/but-api/src/legacy/repo.rs | Renamed function: get_uncommited_files → get_uncommitted_files |
| crates/but-api/src/legacy/oplog.rs | Fixed "commmit" → "commit", "shapshot" → "snapshot" |
| crates/but-api/src/json.rs | Fixed "messsage" → "message" |
| crates/but-action/src/workflow.rs | Fixed "Interupted" → "Interrupted" |
| crates/but-action/src/simple.rs | Fixed "aslo" → "also" |
| crates/but-action/src/lib.rs | Fixed "branche" → "branch", "assingments" → "assignments" |
| crates/but-action/src/generate.rs | Fixed "heirarchy" → "hierarchy" |
| crates/but-action/src/branch_changes.rs | Fixed "exisiting" → "existing" |
| crates/but-action/src/auto_commit.rs | Fixed "exisiting" → "existing" |
| crates/but-action/src/absorb.rs | Fixed "channges" → "changes", "assingments" → "assignments", "branche" → "branch", "ther" → "there", "chages" → "changes" |
| apps/web/src/routes/(app)/[ownerSlug]/[projectSlug]/reviews/[branchId]/+page.svelte | Fixed "origional" → "original" |
| apps/web/src/lib/components/table/TableRow.svelte | Fixed "CHNAGES" → "CHANGES" |
| apps/web/src/lib/components/service/FullscreenIllustrationCard.svelte | Fixed variable name: walkininSvg → walkingSvg (2 instances) |
| apps/web/src/lib/components/rules/RulesModal.svelte | Fixed "aditional" → "additional" |
| apps/web/src/lib/components/review/ReviewSections.svelte | Fixed "paramaters" → "parameters" |
| apps/web/src/lib/components/infoFlexRow/InfoFlexRow.svelte | Fixed "contaner" → "container" |
| apps/desktop/src/routes/onboarding/+page.svelte | Fixed "accessable" → "accessible" |
| apps/desktop/src/routes/[projectId]/history/+page.svelte | Fixed "doesnt" → "doesn't" |
| apps/desktop/src/lib/utils/unsubscribe.test.ts | Fixed "resovles" → "resolves" (3 instances) |
| apps/desktop/src/lib/user/userService.ts | Fixed "loging" → "logging" |
| apps/desktop/src/lib/state/helpers.test.ts | Fixed "precidence" → "precedence" (3 instances), "fufilled" → "fulfilled" (2 instances) |
| apps/desktop/src/lib/state/butlerModule.ts | Fixed "defintion" → "definition" |
| apps/desktop/src/lib/stacks/stackService.svelte.ts | Fixed "accomodate" → "accommodate", "accross" → "across", "acces" → "access" (2 instances) |
| apps/desktop/src/lib/stacks/macros.ts | Fixed "happend" → "happened" |
| apps/desktop/src/lib/soup/commitAnalytics.ts | Fixed "comitting" → "committing" |
| apps/desktop/src/lib/selection/uncommittedService.svelte.ts | Fixed "withing" → "within" |
| apps/desktop/src/lib/selection/uncommitted.ts | Fixed "cooresponding" → "corresponding" (2 instances), "entirly" → "entirely" |
| apps/desktop/src/lib/selection/entityAdapters.ts | Fixed "cooresponds" → "corresponds" |
| apps/desktop/src/lib/notifications/toasts.ts | Fixed "chache" → "cache" |
| apps/desktop/src/lib/irc/ircClient.svelte.ts | Fixed "avaialable" → "available" |
| apps/desktop/src/lib/hunks/change.ts | Fixed "neccessary" → "necessary", "fontend" → "frontend" |
| apps/desktop/src/lib/history/oplogService.svelte.ts | Fixed "Supercedes" → "Supersedes" |
| apps/desktop/src/lib/forge/shared/prFooter.ts | Fixed "fromt" → "from" |
| apps/desktop/src/lib/forge/prPolling.svelte.ts | Fixed "everytime" → "every time" |
| apps/desktop/src/lib/forge/gitlab/types.ts | Fixed "hopfully" → "hopefully" (2 instances) |
| apps/desktop/src/lib/files/filetreeV3.ts | Fixed "tranforming" → "transforming" |
| apps/desktop/src/lib/dragging/registry.ts | Fixed "bascially" → "basically" |
| apps/desktop/src/lib/dependencies/dependencies.ts | Fixed "depdendency" → "dependency", "dependecies" → "dependencies" |
| apps/desktop/src/lib/commits/dropHandler.ts | Fixed "beloning" → "belonging" |
| apps/desktop/src/lib/codegen/messages.ts | Fixed "entirly" → "entirely", "discrepency" → "discrepancy" |
| apps/desktop/src/lib/branches/branchListing.ts | Fixed "uncommited" → "uncommitted", "commited" → "committed", "ther" → "there", "virutal" → "virtual" (2 instances) |
| apps/desktop/src/lib/branches/branch.ts | Fixed "rabased" → "rebased", "afte" → "after" |
| apps/desktop/src/lib/backend/web.ts | Fixed "backedn" → "backend" |
| apps/desktop/src/lib/backend/clipboard.ts | Fixed "failes" → "fails" |
| apps/desktop/src/lib/ai/service.ts | Fixed "specificed" → "specified", "characers" → "characters" |
| apps/desktop/src/lib/ai/prompts.ts | Fixed "assitant" → "assistant" |
| apps/desktop/src/lib/actions/types.ts | Fixed "figgure" → "figure", "interupted" → "interrupted" |
| apps/desktop/src/components/test/readme.md | Fixed "likly" → "likely" |
| apps/desktop/src/components/projectSettings/AppearanceSettings.svelte | Fixed "unassinged" → "unassigned" |
| apps/desktop/src/components/editor/MessageEditor.svelte | Fixed "numver" → "number" |
| apps/desktop/src/components/UnifiedDiffView.svelte | Fixed "isCommiting" → "isCommitting" (2 instances) |
| apps/desktop/src/components/Resizer.svelte | Fixed "adjustement" → "adjustment" |
| apps/desktop/src/components/PullRequestCard.svelte | Fixed "seperator" → "separator" (2 instances) |
| apps/desktop/src/components/NewRuleMenu.svelte | Fixed "catergory" → "category" |
| apps/desktop/src/components/MultiStackView.svelte | Fixed "annoyin" → "annoying" |
| apps/desktop/src/components/IntegrateUpstreamModal.svelte | Fixed "dissapear" → "disappear", "simultaniously" → "simultaneously" |
| apps/desktop/src/components/GithubIntegration.svelte | Fixed "ERRROR" → "ERROR" |
| apps/desktop/src/components/Drawer.svelte | Fixed "everwhere" → "everywhere" |
| apps/desktop/src/components/DraftBranchHeader.svelte | Fixed "commiting" → "committing" |
| apps/desktop/src/components/CommitLineOverlay.svelte | Fixed "advertize" → "advertise" (4 instances) |
| apps/desktop/src/components/ChunkyList.svelte | Fixed "quanity" → "quantity" |
| apps/desktop/src/components/BranchHeader.svelte | Fixed "commiting" → "committing" |
| apps/desktop/cypress/e2e/workspace.cy.ts | Fixed "jsut" → "just" |
| apps/desktop/cypress/e2e/unifiedDiffView.cy.ts | Fixed "expet" → "expect" |
| apps/desktop/cypress/e2e/selection.cy.ts | Fixed "shuold" → "should" (4 instances), "Shouls" → "Should" |
| apps/desktop/cypress/e2e/errorHandling.cy.ts | Fixed "shuold" → "should" (2 instances) |
| apps/desktop/cypress/e2e/branches.cy.ts | Fixed "preivew" → "preview" |
| apps/desktop/README.md | Fixed "shoud" → "should" |
🧢 Changes
Corrected 383 spelling mistakes and grammatical errors across 184 files using systematic code review.
Common fixes:
wth→within test variables,updated_branchesfield name,walkingSvgimportPreserved valid names (false positives):
dateA/dateB- date comparison variablesHasTable- Diesel crate importser- Serde module referenceUpToDate- enum variantratatui- TUI librarywth- log message abbreviation☕️ Reasoning
Spelling errors reduce code professionalism and can cause confusion during code review. The typo
wthin test files was a bug (should referencewithvariable).Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.