-
-
Notifications
You must be signed in to change notification settings - Fork 760
docs: standardize heading case in documentation #13051
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Deploying rspack with
|
| Latest commit: |
65586ba
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://dac8a13e.rspack-v2.pages.dev |
| Branch Preview URL: | https://docs-heading-case-0212.rspack-v2.pages.dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Standardizes Markdown heading capitalization across docs/READMEs and enforces the style via the heading-case CLI in the repo’s JS formatting scripts.
Changes:
- Update heading capitalization in multiple
.md/.mdxfiles to sentence case. - Add
heading-casetoformat:js/format-ci:jsand bump it to^1.0.7. - Move
heading-casedependency management to the root workspace (lockfile updated accordingly).
Reviewed changes
Copilot reviewed 15 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| website/package.json | Removes heading-case from website devDependencies (now managed at repo root). |
| website/docs/zh/guide/migration/rspack_1.x.mdx | Adjusts a section heading to sentence case. |
| website/docs/zh/config/module-rules.mdx | Adjusts page heading to sentence case. |
| website/docs/zh/blog/announcing-1-7.mdx | Adjusts a section heading to sentence case. |
| website/docs/en/guide/tech/rsc.mdx | Adjusts section headings to sentence case. |
| website/docs/en/guide/migration/rspack_1.x.mdx | Adjusts multiple section headings to sentence case. |
| website/docs/en/contribute/development/project.md | Adjusts many headings to sentence case for consistency. |
| website/docs/en/config/module-rules.mdx | Adjusts page heading to sentence case. |
| website/docs/en/config/filename-placeholders.mdx | Adjusts section headings to sentence case. |
| tests/e2e/cases/lazy-compilation/cross-origin/README.md | Adjusts heading capitalization in test documentation. |
| pnpm-lock.yaml | Updates lockfile for [email protected] and importer placement. |
| packages/rspack-test-tools/README.md | Adjusts a section heading to sentence case. |
| package.json | Adds heading-case to formatting scripts and bumps dependency to ^1.0.7. |
| crates/rspack_tools/README.md | Adjusts a command section heading to sentence case. |
| crates/rspack_binding_api/README.md | Adjusts headings to sentence case. |
| AGENTS.md | Adjusts headings to sentence case. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📦 Binary Size-limit
🙈 Size remains the same at 48.68MB |
Rsdoctor Bundle Diff Analysis
Found 5 projects in monorepo, 0 projects with changes. 📊 Quick Summary
Generated by Rsdoctor GitHub Action |
Summary
format:jsscript in package.jsonChecklist