Skip to content

Conversation

leemthompo
Copy link
Contributor

@leemthompo leemthompo commented Sep 30, 2025

Contributes to https://github.com/elastic/docs-content-internal/issues/334

I validated with a full docs-builder build locally

Summary

Created top-level contribute-docs/ section - New main section for public docs contribution guidelines
Moved API docs guidelines from extend/contribute/api-docs/ to contribute-docs/api-docs/
Updated docset.yml - Added toc: contribute-docs to main navigation
Updated extend/toc.yml - Removed references to moved files, simplified structure
Added redirects - All old URLs redirect to new locations
Created new TOC files - contribute-docs/toc.yml and contribute-docs/api-docs/toc.yml

Related PR in docs-builder

elastic/docs-builder#1969

@leemthompo leemthompo self-assigned this Sep 30, 2025
leemthompo added a commit to elastic/docs-builder that referenced this pull request Sep 30, 2025
Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

LGMT — thanks!

Only comment: how about renaming the index "Contribute to the docs" instead of "Contribute to docs"?

@leemthompo leemthompo marked this pull request as ready for review September 30, 2025 14:59
@leemthompo leemthompo requested review from a team as code owners September 30, 2025 14:59
@@ -0,0 +1,11 @@
toc:
- file: index.md
- file: api-docs/index.md
Copy link
Contributor

Choose a reason for hiding this comment

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

  • file: api-docs/index.md

This seems like a duplication of what's in the api-docs/toc.yml file. Is the duplication necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good catch, I actually had a failing all-repo build when I didn't do this duplication

but maybe I could restructure somewhere else?

@leemthompo leemthompo requested review from a team and reakaleek October 2, 2025 10:49
@leemthompo leemthompo merged commit 3bddd62 into main Oct 9, 2025
8 checks passed
@leemthompo leemthompo deleted the leemthompo/contribute-docs-groundwork branch October 9, 2025 12:49
leemthompo added a commit to elastic/docs-builder that referenced this pull request Oct 9, 2025
🐔🥚: elastic/docs-content#3230

Co-authored-by: Fabrizio Ferri-Benedetti <fabri.ferribenedetti@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Initiative 🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants