Add markdownlint configuration and README adjustments#30
Conversation
- Added `.markdownlint.json` for consistent Markdown formatting. - Updated `README.md` with improved badge URLs and new rel attributes. - Adjusted code formatting and disabled specific markdownlint rules where applicable.
|
Warning Rate limit exceeded@AlphaOne1 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 10 minutes and 51 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (2)
WalkthroughA new Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov Report✅ All modified and coverable lines are covered by tests. |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (3)
.markdownlint.json (2)
6-6: Add a newline at EOFEnsure the file ends with a newline to avoid noisy diffs and align with common tooling expectations.
} +
1-6: Optional: Allow HTML elements via MD033 to avoid local disablesIf you prefer not to sprinkle markdownlint-disable comments around the badges, allow specific HTML tags in the config.
{ "default": true, "MD003": { "style": "setext_with_atx" }, "MD007": { "indent": 2 }, - "MD013": { "line_length": 120 } + "MD013": { "line_length": 120 }, + "MD033": { "allowed_elements": ["a", "img", "p", "br"] } }README.md (1)
1-1: markdownlint directives are appropriate for the badge blockTemporarily disabling MD013/MD033/MD041 around the HTML-heavy badge section is a clean approach. If this pattern recurs, consider allowing specific HTML elements via MD033 in the config to avoid local disables.
Also applies to: 71-71
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
.markdownlint.json(1 hunks)README.md(2 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
README.md
189-189: Hard tabs
Column: 1
(MD010, no-hard-tabs)
🔇 Additional comments (2)
.markdownlint.json (1)
1-6: Solid baseline configurationDefaults enabled with targeted overrides for MD003, MD007, MD013 looks good and aligns with the README usage.
README.md (1)
4-39: Badges: security and link updates look goodAdding rel="noopener noreferrer" for target="_blank" links and updating badge URLs (Codecov, GoDoc→pkg.go.dev) are correct. New CodeRabbit badge is fine.
Also applies to: 41-69
Summary by CodeRabbit