Conversation
Made-with: Cursor
📝 WalkthroughWalkthroughA new README best practices cursor rule has been added to the repository. The README.md file references this new rule, and a new Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 277: Move the README rule entry "[README Best
Practices](./rules/readme-best-practices-cursorrules-prompt-file/.cursorrules)"
out of the "### Other" section and add it under the "### Documentation" section
in README.md; specifically, remove the line currently listed under Other and
insert the same markdown list item into the Documentation category so the rule
is documented under the correct heading.
| - [Graphical Apps Development](./rules/graphical-apps-development-cursorrules-prompt-file/.cursorrules) - Cursor rules for graphical apps development with integration. | ||
| - [Meta-Prompt](./rules/meta-prompt-cursorrules-prompt-file/.cursorrules) - Cursor rules for meta-prompt development with integration. | ||
| - [Next.js (Type LLM)](./rules/next-type-llm/.cursorrules) - Cursor rules for Next.js development with Type LLM integration. | ||
| - [README Best Practices](./rules/readme-best-practices-cursorrules-prompt-file/.cursorrules) - Cursor rules for README documentation with best practices integration. |
There was a problem hiding this comment.
Move this rule to the Documentation category.
README Best Practices is documentation-centric and should be listed under ### Documentation, not ### Other, to keep category semantics accurate.
♻️ Proposed fix
@@
-- [README Best Practices](./rules/readme-best-practices-cursorrules-prompt-file/.cursorrules) - Cursor rules for README documentation with best practices integration.
@@
### Documentation
- [Gherkin Style Testing](./rules/gherkin-style-testing-cursorrules-prompt-file/.cursorrules) - Cursor rules for Gherkin style testing development with integration.
- [How-To Documentation](./rules/how-to-documentation-cursorrules-prompt-file/.cursorrules) - Cursor rules for how-to documentation development with integration.
+- [README Best Practices](./rules/readme-best-practices-cursorrules-prompt-file/.cursorrules) - Cursor rules for README documentation with best practices integration.As per coding guidelines: "Update the README.md file when adding new .cursorrules files, placing them in the correct category."
Based on learnings: "Update the README.md file when adding new .cursorrules files, placing them in the correct category."
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` at line 277, Move the README rule entry "[README Best
Practices](./rules/readme-best-practices-cursorrules-prompt-file/.cursorrules)"
out of the "### Other" section and add it under the "### Documentation" section
in README.md; specifically, remove the line currently listed under Other and
insert the same markdown list item into the Documentation category so the rule
is documented under the correct heading.
Adds a cursor rule for writing better READMEs. Agents default to dry, template-style docs with generic openings and bold-label bullet lists. This rule teaches them to write READMEs like landing pages - punchline first, feature tables, copy-paste install blocks.
Source: https://github.com/ofershap/readme-best-practices
Summary by CodeRabbit
Documentation
Chores