Skip to content

docs: add how-to guides for extra dictionaries and pre-commit setup#808

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/duplicate-pr-207
Draft

docs: add how-to guides for extra dictionaries and pre-commit setup#808
Copilot wants to merge 2 commits into
mainfrom
copilot/duplicate-pr-207

Conversation

Copilot AI commented Jun 16, 2026

Copy link
Copy Markdown

PR #207 (by @danielfdickinson) stalled without merging. This duplicates and updates it with all review feedback applied and versions bumped to v10.0.1.

Changes

README.md

  • Replaces the inline "Setup Custom Dictionary" block with a "How-To Guides and Configuration Examples" index

New docs/ pages

  • use-a-custom-dictionary.md — content moved verbatim from README
  • use-dictionaries-from-cspell-dicts.md — adding language dicts via additional_dependencies + import
  • file-or-folder-based-overrides.md — enabling a dictionary per file glob using overrides
  • pre-commit-example-setup-for-french.md — end-to-end French locale example

Review feedback from #207 applied

  • Intro reframed: opens with the default constraint (Bash dict only applies to shellscript files), then shows the fix
  • Inline alternative added: <!--- cspell:dictionaries bash ---> documented as a single-file shortcut
  • pre-commit parity noted inline rather than duplicating a full invocation section
  • Repeated "Invoking CSpell" sections removed; replaced with "See Also" reference lists
  • French example simplified: removed duplicate file content and error-file walkthrough; replaced with a directory tree showing which files match the override

@Jason3S

Jason3S commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

@danielfdickinson,

Would you review the result? Suggestions are welcome. I'll read it a bit later today.

Copilot AI 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.

Pull request overview

This PR restructures cspell-cli documentation by moving configuration examples out of README.md into dedicated docs/ how-to guides, and adds new guides for using extra dictionaries (including cspell-dicts) and pre-commit setups.

Changes:

  • Replace the README’s inline custom-dictionary instructions with a “How-To Guides and Configuration Examples” index.
  • Add new docs covering: custom dictionaries, importing language dictionaries from cspell-dicts, override-based dictionary activation, and a French pre-commit example.
  • Update examples to reference cspell-cli v10.0.1.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
README.md Replaces inline custom dictionary content with links to new docs pages.
docs/use-a-custom-dictionary.md Moves custom dictionary setup guidance into standalone doc.
docs/use-dictionaries-from-cspell-dicts.md Documents adding non-default language dictionaries via additional_dependencies + import.
docs/file-or-folder-based-overrides.md Shows enabling dictionaries by glob/folder using overrides (and an inline directive alternative).
docs/pre-commit-example-setup-for-french.md Provides an end-to-end French dictionary example for pre-commit and cspell config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


To use a language dictionary:

1. Add it as an `additional_dependency` in `.pre-commit-config.yaml`
}
```

If you installed the [Code Spell Checker extension](https://marketplace.visualstudio.com/items?itemName=streetsidesoftware.code-spell-checker) for VS Code, this can be done automatically from the command palette by running "Spell: Create a cSpell configuration file".
@danielfdickinson

Copy link
Copy Markdown

I've only had a few moments for a quick skim, but it is looking good.

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.

4 participants