Skip to content

Conversation

@JohananOppongAmoateng
Copy link
Contributor

@JohananOppongAmoateng JohananOppongAmoateng commented Oct 10, 2025

Description

This pull request updates the documentation to clarify prerequisites for building documentation with spell checking. The main change is the addition of instructions for installing the Enchant library, which is required by the sphinxcontrib-spelling extension.

Documentation improvements:

  • Added a section in docs/contributing.rst detailing how to install the Enchant library for different platforms (Windows, macOS, Linux, FreeBSD), including a link to the official installation guide.

Fixes #2221

Checklist:

  • I have added the relevant tests for this change.
  • I have added an item to the Pending section of docs/changes.rst.

@JohananOppongAmoateng JohananOppongAmoateng force-pushed the docs/add-prerequisites-enchant-install branch from c3650e3 to 64adeeb Compare October 10, 2025 23:26
Comment on lines 228 to 243
Additionally, to build the documentation with proper spell checking,
you need to install:

- **Enchant Library** - This is required by the sphinxcontrib-spelling
extension via the pyenchant package. Installation instructions vary by
platform:

- **Windows**: The binary wheel includes a pre-compiled Enchant library.
- **macOS**: ``brew install enchant``
- **Linux**: Use your distribution's package manager
(e.g., ``apt install enchant``)

- **FreeBSD**: ``pkg install enchant-2``

- **FreeBSD**: ``pkg install enchant-2``

Copy link
Member

Choose a reason for hiding this comment

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

I think we should try to be more concise here. We're pretty far removed from enchant, so I think forwarding people to pyenchant's documentation is sufficient rather than trying to explain all the different methods. Does that seem reasonable to you?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes i understand thats true

@JohananOppongAmoateng JohananOppongAmoateng force-pushed the docs/add-prerequisites-enchant-install branch from db03887 to 13fc5c0 Compare October 12, 2025 15:55
Copy link
Member

@tim-schilling tim-schilling left a comment

Choose a reason for hiding this comment

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

Thank you!

@tim-schilling tim-schilling merged commit 85885d7 into django-commons:main Oct 13, 2025
25 of 29 checks passed
@tim-schilling
Copy link
Member

Thank you @okotdaniel for identifying this issue!

@okotdaniel
Copy link
Member

Thank you @okotdaniel for identifying this issue!

You are welcome.

@JohananOppongAmoateng JohananOppongAmoateng deleted the docs/add-prerequisites-enchant-install branch October 13, 2025 20:16
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.

Document the enchant dependency for building docs

3 participants