-
Notifications
You must be signed in to change notification settings - Fork 7
Update README.md #6
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
Conversation
WalkthroughREADME.md updated to add a new "Project Structure" section containing an ASCII map of the repository layout; the section is placed after Architecture and before Quick Start. This is a documentation-only change with no code, runtime, or public API modifications. Changes
Pre-merge checks✅ Passed checks (5 passed)
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. 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: 1
🧹 Nitpick comments (1)
README.md (1)
91-91: Add a language to the fenced code block to satisfy markdownlint (MD040).Specify the fence as "text" (or "plaintext") so linters don’t warn and GitHub renders appropriately.
-``` +```text
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(2 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
README.md
91-91: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🔇 Additional comments (2)
README.md (2)
90-125: Nice addition — clear, beginner-friendly structure.The emojis and concise folder comments improve scannability.
90-125: Add top-level ui/ or update README's Architecture sectionREADME.md (lines 90–125) references a top-level ui service but the repository has no ui/ directory; the verification script reported no other missing top-level entries. Either add the ui/ service or remove/update that entry in README.md.
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: 0
♻️ Duplicate comments (1)
README.md (1)
92-92: Root label fixed — looks correct now.Tree root shows "📦 DialogWeaver" (matches repo). Good catch and fix.
🧹 Nitpick comments (2)
README.md (2)
91-91: Add a language to the fenced code block (markdownlint MD040).Specify a language for the project tree block to satisfy linters and improve rendering.
-``` +```text
90-125: Optional: Add quick links below the tree.Add short bullet links to frequently used files (e.g.,
api/main.py,engine/main.py,docker-compose.yml) right under the code block for faster navigation.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(2 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
README.md
91-91: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🔇 Additional comments (1)
README.md (1)
97-117: Directory tree verified — all listed paths exist.
Requested paths (including api/*, engine/openvoiceos_engine, engine/telephony_server, and docs assets) are present.
Related Issue(s)
Fixes #5
##Summary##
The README currently lacked a clear explanation of the project’s folder structure. This PR adds a detailed project structure section with descriptions for all main folders and files, making it easier for new contributors to understand the codebase and navigate the project.
##Changes##
Added a Project Structure section to the README.
Included descriptions for key folders and files: .github, api, docs, engine, and root files.
Structured the documentation so it is clear, readable, and beginner-friendly.
##How to Test##
Open the updated README.md.
Verify that the Project Structure section is present and matches the actual repo layout.
Ensure that all descriptions are accurate and readable.
Note: Please ensure that appropriate labels (like gssoc25 and level labels) are assigned to the merged PR.
Summary by CodeRabbit