Repository navigation
chore(website): remove unused util, CSS classes and CSS module (AI-assisted) - #12766
Merged
Merged
Conversation
Only used by its own test since the showcase refactor (#10023). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ssisted) These CSS module classes are not used by any component anymore, and the React MDX docs page imported a CSS module it doesn't use since the MDX v2 docs rewrite (#8941). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
slorber
added
pr: maintenance
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation |
|---|---|---|
| Description Check |  |
Check skipped - CodeRabbit’s high-level summary is enabled. |
| Title check | |
The title "chore(website): remove unused util, CSS classes and CSS module (AI-assisted)" matches the main change. The PR removes the unused toggleListItem utility and its test, several unreferenced CS… |
| Docstring Coverage | |
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Title check
-
Explanation
The title "chore(website): remove unused util, CSS classes and CSS module (AI-assisted)" matches the main change. The PR removes the unused toggleListItem utility and its test, several unreferenced CSS classes, and the React Markdown-features CSS module and its import. The title names the website scope and the type of change. The "(AI-assisted)" suffix is extra detail, not a content error. The title is short and specific, so it passes.
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.
✨ Finishing Touches
-
📝 Generate docstrings
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
✅ [V2]
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Size Change: -680 B (-0.01%) Total Size: 12.1 MB 📦 View Changed
ℹ️ View Unchanged
|
⚡️ Lighthouse report for the deploy preview of this PR
|
|
Size Change: -680 B (-0.01%) Total Size: 12.1 MB 📦 View Changed
ℹ️ View Unchanged
|
|
The latest updates on your projects. Learn more about Argos notifications ↗︎
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Remove dead code from the Docusaurus website (
website/). No visual change.src/utils/jsUtils.ts: thetoggleListItemutil, and its test filesrc/utils/__tests__/jsUtils.test.ts, which only tests it. Only its test uses it since the showcase refactor (refactor(website): refactor showcase components #10023).sortBystays.styles[...]access either):src/pages/styles.module.css:.sectionInner,.announcement,.announcementDark,.announcementInner,.indexCtaTryNowButton:hoversrc/components/BrowserWindow/styles.module.css:.row::after,.rightsrc/plugins/changelog/theme/ChangelogItem/Header/Authors/styles.module.css:.toggleButtonIconExpanded(theIconExpandcomponent handles the expanded state)blog/2022/08-01-announcing-docusaurus-2.0/ShowcaseCarousel.module.css:.siteSlidedocs/guides/markdown-features/markdown-features-react.module.css, and its unusedimport stylesinmarkdown-features-react.mdx, unused since the MDX v2 docs rewrite (docs: rewrite some docs for mdx v2 #8941).versioned_docs/is left untouched.This PR was written with AI assistance (AI-assisted).
Argos
Expected: no screenshot changes, but the
styles.csstext snapshot changes and needs approval:@layer docusaurus.theme-classic. The CSS bundle order changes because the React MDX page no longer imports its CSS module. These rules target different elements, so their relative order has no visual effect.Test Plan
rg -afinds no reference to the removed names inwebsite/(outsideversioned_docs/andbuild/)pnpm test websitepassespnpm --filter website typecheckpassespnpm lint:style,pnpm lint:knipand oxfmt passTest links
Deploy preview: https://deploy-preview-12766--docusaurus-2.netlify.app/
Related issues/PRs
🤖 Generated with Claude Code
Summary by CodeRabbit