-
Notifications
You must be signed in to change notification settings - Fork 15
Updated the headings and the information style #101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Updated the headings and the information style #101
Conversation
|
|
WalkthroughDocumentation content in docs/get-started/intro.md was revised: headings retitled, feature descriptions expanded, limitations clarified, code-writing phrasing adjusted, and audience section extended with examples. No code or behavioral changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
🧹 Nitpick comments (2)
docs/get-started/intro.md (2)
26-26: Tone/claims: prefer precise, neutral phrasing.
Suggest avoiding “highly creative tool.” Keep claims factual about testing and media capture.-Doc Detective is a highly creative tool that can perform documentation testing and generate screenshots to support your documentation. +Doc Detective performs documentation testing and can capture screenshots to support your documentation.
1-4: Consider migrating this page to .mdx (project guideline).
The repo guideline says “docs/**/*.md: Do not add .md files; use .mdx instead.” While this is an edit to an existing .md, consider renaming to intro.mdx and updating inbound links to align with the standard.As per coding guidelines
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/get-started/intro.md(2 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
docs/**/*.md
📄 CodeRabbit inference engine (AGENTS.md)
Do not add .md files for docs; use .mdx instead
Files:
docs/get-started/intro.md
🧠 Learnings (1)
📚 Learning: 2025-10-06T23:29:55.100Z
Learnt from: CR
PR: doc-detective/doc-detective.github.io#0
File: AGENTS.md:0-0
Timestamp: 2025-10-06T23:29:55.100Z
Learning: Applies to **/*.spec.json : Define Doc Detective tests in .spec.json files
Applied to files:
docs/get-started/intro.md
🔇 Additional comments (4)
docs/get-started/intro.md (4)
18-18: Heading update looks good.
Clear, user-friendly section title.
24-24: Section title change is appropriate.
Accurately groups feature content.
43-43: Limitation section heading LGTM.
Accurate and clear.
55-55: Audience heading LGTM.
Clear and consistent with the new structure.
| ## Current Limitations | ||
|
|
||
| It’s important to know your limits, and Doc Detective’s too. | ||
| Doc Detective has it's own set of limitations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grammar: “its” (possessive), not “it’s.”
-Doc Detective has it's own set of limitations.
+Doc Detective has its own set of limitations.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Doc Detective has it's own set of limitations. | |
| Doc Detective has its own set of limitations. |
🤖 Prompt for AI Agents
In docs/get-started/intro.md around line 45, the phrase "Doc Detective has it's
own set of limitations." uses the contraction "it's" incorrectly; change "it's"
to the possessive "its" so the sentence reads "Doc Detective has its own set of
limitations." to fix the grammar.
| ### Write your code | ||
|
|
||
| While Doc Detective can write tests from scanning your documentation, it can’t write code for you. Sorry. | ||
| Doc Detective can only write test cases by scanning your documentation; it doesn't write code. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correctness: Doc Detective doesn’t write tests; users define them.
Current sentence implies auto‑generating tests from docs. Tests are defined in .spec.json files and executed by Doc Detective.
-Doc Detective can only write test cases by scanning your documentation; it doesn't write code.
+Doc Detective doesn't write code or generate tests. You define tests (for example, in .spec.json files), and Doc Detective executes them against your product.Based on learnings
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Doc Detective can only write test cases by scanning your documentation; it doesn't write code. | |
| Doc Detective doesn't write code or generate tests. You define tests (for example, in .spec.json files), and Doc Detective executes them against your product. |
🤖 Prompt for AI Agents
In docs/get-started/intro.md around line 53, the sentence "Doc Detective can
only write test cases by scanning your documentation; it doesn't write code."
incorrectly implies Doc Detective auto-generates tests from docs; update this
line to state that Doc Detective does not write tests automatically and that
users must define tests in .spec.json files which Doc Detective reads and
executes (i.e., rephrase to clarify that tests are authored by users in
.spec.json and Doc Detective runs them, not that it generates them).
|
Thanks for your contribution! We're actively refactoring our Vale rules, so we're holding off on any style-based PRs for the moment. We'll reevaluate once the new rules are in place. |
I have also rephrased some of the explanations, please have a look at those changes. |
Description
This PR focuses on headings and information restructuring. I updated the introduction sections as per the tech writing standards.
Type of contribution
Related issue
Closes # Related to #
Changes made
Testing performed
npm run start)npm run build)Screenshots (if applicable)
Checklist
.mdxextension/docs/pagenot../page)Additional notes
For reviewers:
Summary by CodeRabbit