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
rhr323 pushed a commit to rhr323/docs-content that referenced this pull request Oct 27, 2025
…ic#3230)

Contributes to
elastic/docs-content-internal#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

---------

Co-authored-by: Fabrizio Ferri-Benedetti <[email protected]>
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.

7 participants