Skip to content

docs: fix spelling checks for control write gate - #5514

Merged
springfall2008 merged 1 commit into
feat/control-write-drain-gatefrom
fix/control-gate-spelling
Oct 12, 2026
Merged

springfall2008 merged 1 commit into
feat/control-write-drain-gatefrom
fix/control-gate-spelling

Conversation

@mgazza

@mgazza mgazza commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator

The new control-gate documentation and test keyword failed the repository cspell hook. Add the standard Python identifiers fromlist and PYTHONPATH to the sorted custom dictionary and replace one unfamiliar documentation word with plain wording.

Validation: all configured pre-commit hooks pass on the engine patch files, including cspell, dictionary sorting, Black, Ruff and Markdown lint. No runtime code changes. Stacked on #5513 and required with it.

@springfall2008
springfall2008 marked this pull request as ready for review October 12, 2026 17:29
Copilot AI balanced review requested due to automatic review settings October 12, 2026 17:29
@springfall2008
springfall2008 merged commit d15e054 into feat/control-write-drain-gate Oct 12, 2026
3 checks passed
@springfall2008
springfall2008 deleted the fix/control-gate-spelling branch October 12, 2026 17:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The documentation-only changes are correct, consistently sorted, and introduce no runtime impact.

0 open findings

What changed in this PR

Updates spelling-check inputs for the stacked control-write gate documentation and tests.

Changes:

  • Adds fromlist and PYTHONPATH to the custom dictionary.
  • Rephrases “ungated controller” in plain language.
File Description
.cspell/​custom-dictionary-workspace.txt Adds sorted spelling exceptions.
docs/​control-write-drain-gate.md Rewords an unfamiliar term.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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.

3 participants