Skip to content

Exclude Hugo shortcodes from spellchecking - #276

Merged
NValsan merged 1 commit into
drasi-project:mainfrom
NValsan:fix/spellcheck-hugo-shortcodes
Sep 17, 2026
Merged

NValsan merged 1 commit into
drasi-project:mainfrom
NValsan:fix/spellcheck-hugo-shortcodes

Conversation

@NValsan

@NValsan NValsan commented Sep 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Run a PySpelling context filter before Markdown processing to exclude both Hugo shortcode syntaxes.
  • Match multiline shortcodes while preserving surrounding prose and content between paired shortcodes.
  • Leave existing dictionaries, filters, and encoding settings unchanged.

Validation

  • Passed six direct PySpelling filter checks covering both syntaxes, multiline parameters, and preserved prose.
  • git diff --check passed.
  • Full repository spellcheck was not run.

@NValsan
NValsan requested review from a team and a lite review from Copilot September 15, 2026 00:33

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

No unresolved review issues were identified.

Pull request overview

Updates PySpelling to exclude Hugo shortcodes before Markdown processing.

Changes:

  • Adds filters for both Hugo shortcode syntaxes.
  • Supports multiline shortcode parameters.
  • Preserves existing spellcheck settings.
File summaries
File Description
.github/config/.pyspelling.yml Adds Hugo shortcode context filtering.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

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

@NValsan NValsan self-assigned this Sep 16, 2026
@NValsan
NValsan merged commit 972c3be into drasi-project:main Sep 17, 2026
4 checks passed
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