Skip to content

add Canonical acronyms and terms to spelling exceptions#367

Merged
akcano merged 1 commit into
canonical:mainfrom
akcano:main
Mar 28, 2025
Merged

add Canonical acronyms and terms to spelling exceptions#367
akcano merged 1 commit into
canonical:mainfrom
akcano:main

Conversation

@akcano

@akcano akcano commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

This adds acronyms and terms mined from our glossaries as two separate sublists in wordlist.txt for convenience.

  • Have you updated CHANGELOG.md with relevant file changes?
  • Have you updated the documentation for this change?

This adds acronyms and terms mined from our glossaries
as two separate sublists in wordlist.txt for convenience.
@akcano akcano requested a review from SecondSkoll March 27, 2025 09:31
@akcano akcano self-assigned this Mar 27, 2025
@SecondSkoll

SecondSkoll commented Mar 27, 2025

Copy link
Copy Markdown
Collaborator

This should be going into canonical/praecepta where the global rules and wordlist are defined.

Edit: Well... Actually this is used as well. Do you think it's better to have it here and to merge the wordlist from there to here? The list should be in a single place.

@akcano

akcano commented Mar 28, 2025

Copy link
Copy Markdown
Contributor Author

TBH I'd drop the current HTML-based spellcheck altogether (I actually tried to do so previously but failed because reasons). Syndicating it from here doesn't seem sustainable or warranted; let's just add the lists there as well.

@SecondSkoll

Copy link
Copy Markdown
Collaborator

The way Vale works is that it will convert to HTML to check things as well, to check classes and scopes. It doesn't use Sphinx though, so it's definitely better.

Good plan, lets add it to both, then single source once pyspelling is removed.

@akcano akcano merged commit ae15d5b into canonical:main Mar 28, 2025
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.

2 participants