Skip to content

feat: sanitized fetched docs#140

Merged
avivkeller merged 2 commits into
mainfrom
feat/sanitize-fetched-markdown
Jun 16, 2026
Merged

feat: sanitized fetched docs#140
avivkeller merged 2 commits into
mainfrom
feat/sanitize-fetched-markdown

Conversation

@ryzrr

@ryzrr ryzrr commented Jun 16, 2026

Copy link
Copy Markdown
Member

Links : #138 & used the regex to solve this !

Summary

Moves the cleanup for fetched docs (loader/plugin READMEs and governance) into one shared util, so both scripts run the same thing instead of each doing half of it.
It rewrites relative links to absolute, strips badges, and drops the Contributing/License footers so the pages read like docs.

Copilot AI review requested due to automatic review settings June 16, 2026 16:34

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vercel

vercel Bot commented Jun 16, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
webpack-doc-kit Ready Ready Preview, Comment Jun 16, 2026 5:26pm

Request Review

@ryzrr

ryzrr commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

tried the regex approach, but if needed i can shift to remark based

Comment thread scripts/utils/sanitize.mjs Outdated
Comment thread scripts/utils/sanitize.mjs Outdated

@avivkeller avivkeller left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, see nitpicks

@ryzrr

ryzrr commented Jun 16, 2026

Copy link
Copy Markdown
Member Author

LGTM, see nitpicks

okay , accepting the nitpicks ,btw the final version is much more cleaner and readable ,Thanks !

@avivkeller avivkeller merged commit c20526b into main Jun 16, 2026
4 checks passed
@ryzrr ryzrr deleted the feat/sanitize-fetched-markdown branch June 16, 2026 17:37
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