#22141 closed Cleanup/optimization (fixed)

Add spell checker for documentation

Reported by: beregond Owned by: timo
Component: Documentation Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Spell checker would find all typos in documentation in automatic way. Thanks to it people will be able to automatically check if all they wrote are proper words.

There already exists PR for that: https://github.com/django/django/pull/2354

Change History (6)

comment:1 follow-up: Changed 14 months ago by EvilDMP

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

This is neat, but as bmispelon asked, who is going to be curating - i.e. maintaining - the list?

It's a spelling checker by the way. A spell checker would be useful for a wizard. (I'm aware that this particular boat sailed long ago...)

comment:2 Changed 14 months ago by timo

  • Has patch set
  • Owner changed from beregond to timo
  • Status changed from new to assigned
  • Summary changed from Add spell checker to Add spell checker for documentation
  • Triage Stage changed from Unreviewed to Accepted

I think I would probably be willing to look after this. I will try to review the PR soon.

comment:3 Changed 14 months ago by beregond

EvilDMP - perfectly there is no need to add new words, but if it happens that someone adds word to docs, that is not recognized, and he notices the mistake, he should just add this word to wordlist, so then it is clear (in PR) that he added particular word. In the end developers that make changes SHOULD maintain the wordlist. (It is possible to add this check as test, so if someone will modify docs and this test will fail, he will either fix his typo or add this word to whitelist.)

comment:4 in reply to: ↑ 1 Changed 14 months ago by russellm

Replying to EvilDMP:

It's a spelling checker by the way. A spell checker would be useful for a wizard. (I'm aware that this particular boat sailed long ago...)

Yes. Magic removal landed almost 8 years ago. Spell checking is no longer required.

(/rimshot)

comment:5 Changed 14 months ago by timo

  • Patch needs improvement set

comment:6 Changed 14 months ago by Tim Graham <timograham@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

In a0f252520291924f8fb7cb0d85f1680294508560:

Fixed #22141 -- Added a spelling checker for documentation.

Note: See TracTickets for help on using tickets.
Back to Top