Skip to content

Conversation

@sharadregoti
Copy link
Contributor

@sharadregoti sharadregoti commented Oct 6, 2025

PR Type

Enhancement, Documentation


Description

  • Remove unused MDX snippet components

  • Clean up badges, grids, calculators

  • Update href rewriting regex comment

  • Reduce snippet footprint and clutter


Diagram Walkthrough

flowchart LR
  Snippets["Remove MDX snippets"] -- delete --> Badge["BadgeCard, ButtonLeft, FeatureCards"]
  Snippets -- delete --> Grids["Grid, ResponsiveGrid, UIExampleCards"]
  Snippets -- delete --> Calculators["Redis/Database calculators"]
  Snippets -- delete --> Misc["GitHubStarButton, RedisVersions, ReleaseSummary"]
  Script["merge_docs_configs.py"] -- clarify --> HrefRegex["Comment: BadgeCard not used"]
Loading

File Walkthrough

Relevant files
Documentation
9 files
merge_docs_configs.py
Clarify href rewrite regex usage note                                       
+1/-0     
AIStudioCards.mdx
Remove AI Studio feature cards snippet                                     
+0/-26   
ButtonLeft.mdx
Remove ButtonLeft CTA component                                                   
+0/-54   
DatabaseCalculator.mdx
Remove database storage calculator stub                                   
+0/-5     
GitHubStarButton.mdx
Remove GitHub star button snippet                                               
+0/-16   
Grid.mdx
Remove GridWrapper layout component                                           
+0/-19   
RedisCalculator.mdx
Remove Redis RAM calculator stub                                                 
+0/-6     
RedisVersions.mdx
Remove Redis supported versions snippet                                   
+0/-4     
UIExampleCards.mdx
Remove UI example cards snippet                                                   
+0/-34   
Enhancement
4 files
Badge.mdx
Remove BadgeCard component implementation                               
+0/-36   
FeatureCards.mdx
Remove FeatureCards grid component                                             
+0/-18   
ReleaseSummaryComponent.mdx
Remove release summary component and grid usage                   
+0/-67   
ResponsiveGrid.mdx
Remove responsive grid component                                                 
+0/-7     

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Typo in Comment

The added comment says "BadgeCard is not beign used" which contains a typo; ensure the comment is corrected for clarity and professionalism.

# BadgeCard is not beign used
content = re.sub(r'<(Card|BadgeCard|Expandable)([^>]*\s+)href="([^"]+)"([^>]*>)', replace_card_href, content)
Comment-Regex Mismatch

The regex still includes BadgeCard while the comment claims it's not used; verify whether BadgeCard should remain in the pattern or the comment should be adjusted to avoid confusion.

# BadgeCard is not beign used
content = re.sub(r'<(Card|BadgeCard|Expandable)([^>]*\s+)href="([^"]+)"([^>]*>)', replace_card_href, content)

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
General
Correct comment typo

Fix the typo in the comment and ensure it doesn't mislead future maintainers. Typos
in comments can cause confusion about intended behavior and usage.

scripts/merge_docs_configs.py [1130-1131]

-# BadgeCard is not beign used
+# BadgeCard is not being used
 content = re.sub(r'<(Card|BadgeCard|Expandable)([^>]*\s+)href="([^"]+)"([^>]*>)', replace_card_href, content)
Suggestion importance[1-10]: 2

__

Why: The suggestion correctly fixes a typo in the comment (beign -> being) without altering code behavior, improving readability slightly. Impact is minimal and purely cosmetic.

Low

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants