Skip to content

Conversation

@Karthikg998
Copy link

@Karthikg998 Karthikg998 commented Oct 4, 2025

Title:
Add Setup section for installing dev dependencies and doc-builder

Description:
This PR adds a Setup section to docs/README.md that guides contributors on installing the necessary development dependencies and the Hugging Face doc-builder tool. This makes it easier for new contributors to generate and preview documentation locally.

Motivation & Context:

Improves contributor onboarding with clear setup instructions.

Ensures the documentation can be built locally without errors.

Checklist:

Improves documentation.

Follows contributor guidelines.

No new tests are required (documentation-only change).

Who can review:
Any contributor familiar with documentation or the docs setup.
@zucchini-nlp tagging for prioritising review

Add Setup section for installing dev dependencies and doc-builder
@Karthikg998
Copy link
Author

Karthikg998 commented Oct 6, 2025

Hi @gante @zucchini-nlp could you please review this PR when you have a chance? It adds the Setup section to the README for dev dependencies and doc-builder. Thanks!

Comment on lines +19 to +21
```bash
pip install -e ".[dev]" # Installs dev dependencies
pip install git+https://github.com/huggingface/doc-builder # Installs doc-builder
Copy link
Member

Choose a reason for hiding this comment

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

hmm, i think this is a bit redundant as the package installation is detailed in lines 23 and 45 in main branch

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@Karthikg998 Karthikg998 closed this Oct 6, 2025
@Karthikg998 Karthikg998 deleted the docs-fix-bert branch October 6, 2025 11:00
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