Skip to content

DOC: Sync spell-check dictionary with main branch#6056

Merged
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:release-5.4from
hjmjohnson:backport-dictionary
Apr 14, 2026
Merged

DOC: Sync spell-check dictionary with main branch#6056
hjmjohnson merged 1 commit intoInsightSoftwareConsortium:release-5.4from
hjmjohnson:backport-dictionary

Conversation

@hjmjohnson
Copy link
Copy Markdown
Member

Backport the itk_dict.txt word list from main to release-5.4, adding 433 words accumulated through new features and classes. Uses main's sort order to minimize future merge diffs. Part of #6051.

No behavioral change — the dictionary is a comment spell-checker allowlist only.

@github-actions github-actions bot added type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots type:Documentation Documentation improvement or change labels Apr 14, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 14, 2026

Greptile Summary

This PR backports the itk_dict.txt spell-check allowlist from main to release-5.4, adding approximately 433 domain-specific terms accumulated through new features and classes. There is no behavioral change — the file is used solely as a comment spell-checker allowlist.

Confidence Score: 5/5

  • Safe to merge — documentation-only change with no code logic or behavioral impact.
  • The change is limited to a single spell-checker allowlist file. No code, build logic, or configuration is modified. All findings are P2 or lower (none exist here), so a score of 5 is appropriate.
  • No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/itk_dict.txt Spell-check allowlist updated by backporting 433 words from the main branch to release-5.4. No code logic or behavioral change — purely a dictionary sync.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[main branch\nitk_dict.txt\n~433 new words] -->|backport| B[release-5.4\nitk_dict.txt]
    B --> C{Spell-checker\nCI workflow}
    C -->|allowlist lookup| D[Comment / Doxygen\nspell check passes]
Loading

Reviews (1): Last reviewed commit: "DOC: Sync spell-check dictionary with ma..." | Re-trigger Greptile

@hjmjohnson hjmjohnson force-pushed the backport-dictionary branch 2 times, most recently from 6240e78 to b7bbaf7 Compare April 14, 2026 12:09
Backport the itk_dict.txt word list from upstream/main to release-5.4,
then add 20 words found in release-5.4 source that are not in main's
dictionary (code identifiers in ThirdParty headers, legitimate terms
like Hounsfield and Dask, and variable names in comments that cannot
be changed on the release branch).

Uses main's sort order to minimize future merge diffs.
@hjmjohnson hjmjohnson force-pushed the backport-dictionary branch from b7bbaf7 to 26b48ed Compare April 14, 2026 12:17
@hjmjohnson hjmjohnson merged commit a0892bf into InsightSoftwareConsortium:release-5.4 Apr 14, 2026
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:Documentation Documentation improvement or change type:Infrastructure Infrastructure/ecosystem related changes, such as CMake or buildbots

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants