Skip to content

STYLE: Upgrade to comment_spell_check v0.4.0 #2300

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
dave3d merged 1 commit into from
May 28, 2025
Merged

STYLE: Upgrade to comment_spell_check v0.4.0 #2300

dave3d merged 1 commit into from
May 28, 2025

Conversation

@dave3d
Copy link
Member

@dave3d dave3d commented May 23, 2025

Upgraded to new pure Python spell checking. Also added some words to the dictionary and re-sorted it.

@kwrobot-v1
Copy link

kwrobot-v1 bot commented May 23, 2025

Errors:

  • Failed to fetch from the repository: host error: no result after exponential backoff.
  • Failed to reserve ref 09e52bf for the merge request: invalid git ref: 'no such commit'.
  • Failed to run the checks: mr utilities error: failed to list commits of 09e52bf0df979512c2e7d4c5538331ec10e45846 for https://github.com/SimpleITK/SimpleITK/pull/2300: fatal: bad object 09e52bf0df979512c2e7d4c5538331ec10e45846 .

@dave3d
Copy link
Member Author

dave3d commented May 23, 2025

Errors:

  • Failed to fetch from the repository: host error: no result after exponential backoff.
  • Failed to reserve ref 09e52bf for the merge request: invalid git ref: 'no such commit'.
  • Failed to run the checks: mr utilities error: failed to list commits of 09e52bf0df979512c2e7d4c5538331ec10e45846 for https://github.com/SimpleITK/SimpleITK/pull/2300: fatal: bad object 09e52bf0df979512c2e7d4c5538331ec10e45846 .

@blowekamp What the heck does this mean?

@dave3d
STYLE: Upgrade to comment_spell_check v0.4.0
bfc3795 
Upgraded to new pure Python spell checking.  Also added some
words to the dictionary and re-sorted it.
@dave3d dave3d force-pushed the UpgradeSpelling branch from 3a79154 to bfc3795 Compare May 28, 2025 15:33
@dave3d dave3d requested review from blowekamp and zivy May 28, 2025 15:38
@dave3d dave3d merged commit cb5ca75 into SimpleITK:master May 28, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blowekamp blowekamp blowekamp approved these changes

@zivy zivy zivy 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

@dave3d @blowekamp @zivy