Repository navigation
docs: fix incorrect CommonJS usage heading - #1673
Olexandr88 wants to merge 1 commit into
Conversation
WalkthroughDocumentation-only change: the README's Usage section header in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryFixes the heading in the
Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant U as User
participant R as README.md
participant E as Example File (request-client-js.ts)
U->>R: Reads Usage section
R->>R: Heading: "Usage (ESM / TypeScript)"
R->>E: Links to TypeScript example
E-->>U: Shows ESM import syntax
|
There was a problem hiding this comment.
Additional Comments (1)
-
packages/request-client.js/README.md, line 18-20 (link)style: Extra blank line introduced after the heading. Consider removing for consistency.
1 file reviewed, 1 comment
f4074a2 to
09b8be9
Compare
|
Closing as incorrect. |
Description of the changes
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.