Skip to content

Validate community page metadata - #298

Merged
fc-pinky merged 1 commit into
mainfrom
validate-community-metadata
Sep 29, 2026
Merged

fc-pinky merged 1 commit into
mainfrom
validate-community-metadata

Conversation

@joaomlneto

@joaomlneto joaomlneto commented Jul 7, 2026 •

Copy link
Copy Markdown
Member

Add front-matter validation for community pages so missing or malformed type, title, description, or maintainer fields are surfaced instead of silently dropping pages from the tables. Also make the community page sort key tolerant of non-string titles.

This PR's CI run before #297 and #296 failed: https://github.com/esi/esi-docs/actions/runs/28849384799/job/85560737442
This PR's CI run after including those fixes succeeds: https://github.com/esi/esi-docs/actions/runs/28859477763/job/85594195011

@joaomlneto
joaomlneto requested review from a team as code owners July 7, 2026 07:30
@joaomlneto
joaomlneto force-pushed the validate-community-metadata branch from d55ac5b to ebbe6c9 Compare July 7, 2026 10:29
@fc-pinky
fc-pinky force-pushed the validate-community-metadata branch 2 times, most recently from 52bb790 to 8be5700 Compare September 29, 2026 09:51
Add front-matter validation for community pages so missing or malformed `type`, `title`, `description`, or maintainer fields are surfaced instead of silently dropping pages from the tables.
Also make the community page sort key tolerant of non-string titles.
@fc-pinky
fc-pinky force-pushed the validate-community-metadata branch from 8be5700 to 1fd6486 Compare September 29, 2026 09:57

@fc-pinky fc-pinky left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for this!

@fc-pinky
fc-pinky enabled auto-merge (squash) September 29, 2026 09:57
@fc-pinky
fc-pinky merged commit b059d93 into main Sep 29, 2026
1 check passed
@fc-pinky
fc-pinky deleted the validate-community-metadata branch September 29, 2026 09:58
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.

2 participants