-
Notifications
You must be signed in to change notification settings - Fork 954
fix: simplify installation in readme #577
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
WalkthroughThe README document has been updated to improve formatting, clarity, and structure. The changes include removal of Changes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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
🧹 Nitpick comments (2)
README.md (2)
185-207: Database-Specific Installation Instructions:
The new section on installing Cognee with specific database support is comprehensive and well-structured, offering clear pip-based commands and examples.
- Note: A static analysis hint flagged a possible duplication or repetition in this section. It may be worthwhile to double-check the phrasing for smoothness and clarity.
🧰 Tools
🪛 LanguageTool
[duplication] ~185-~185: Possible typo: you repeated a word.
Context: ...# Install Cognee with specific database support Support for various databases and vector stores...(ENGLISH_WORD_REPEAT_RULE)
246-249: Contributing Section Update:
The contributing guidelines are now highlighted more prominently. For improved grammatical accuracy, consider hyphenating "open source project" to "open‑source project" when used as a compound adjective.🧰 Tools
🪛 LanguageTool
[uncategorized] ~248-~248: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...s are at the core of making this a true open source project. Any contributions you make are...(EN_COMPOUND_ADJECTIVE_INTERNAL)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
assets/cognee-logo-transparent.pngis excluded by!**/*.pngassets/cognee_logo_transparent.pngis excluded by!**/*.png
📒 Files selected for processing (1)
README.md(6 hunks)
🧰 Additional context used
🪛 LanguageTool
README.md
[duplication] ~185-~185: Possible typo: you repeated a word.
Context: ...# Install Cognee with specific database support Support for various databases and vector stores...
(ENGLISH_WORD_REPEAT_RULE)
[uncategorized] ~248-~248: If this is a compound adjective that modifies the following noun, use a hyphen.
Context: ...s are at the core of making this a true open source project. Any contributions you make are...
(EN_COMPOUND_ADJECTIVE_INTERNAL)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: lint (ubuntu-latest, 3.10.x)
- GitHub Check: Build Cognee Backend Docker App Image
- GitHub Check: docker-compose-test
🔇 Additional comments (13)
README.md (13)
1-3: Logo and Layout Update:
The updated<div>now centers content using thestyleattribute, and the logo image (with a new source and reduced height) improves visual consistency over the previous version.
8-18: Enhanced Brand Messaging and Social Badges:
The updated tagline ("cognee - memory layer for AI apps and Agents") and the refreshed social badges clearly communicate Cognee’s identity and mission.
21-21: Header Level Adjustment:
Changing "What is cognee?" from a secondary (##) to a primary (#) header strengthens the semantic hierarchy of the document.
34-37: Benefits Diagram Inclusion:
The centered benefits image (with its associated<div>) effectively enhances visual appeal. Consider evaluating whether thealttext ("Why cognee?") might be made more descriptive to aid accessibility.
41-41: Installation Overview Clarity:
The revised sentence now succinctly lists the supported package managers (pip, poetry, uv, etc.), which aids readability and aligns well with the goal of simplifying the installation instructions.
51-53: Poetry Installation Update:
The updated Poetry installation command (poetry add cognee) simplifies the process and removes unnecessary complexity, meeting the objective of streamlining instructions.
55-58: UV Package Manager Support:
Introducing the installation command for uv (uv add cognee) extends support for alternative package managers, catering to diverse developer preferences.
123-123: Example Usage Clarification:
The updated query example (settingquery_textin the sample script) offers a clearer demonstration of search functionality.
152-159: Architecture Section Enhancement:
Revising the "Understand our architecture" header and its accompanying description—as well as incorporating a centered diagram—improves the overall clarity and organization of the documentation.
208-225: Poetry Extra Installation Instructions:
The additions for installing Cognee with extras using Poetry (including clear commands and examples) are both helpful and clearly presented.
226-235: Local Development Section:
The "Working with local Cognee" section now provides clear, step-by-step instructions for setting up the local development environment, which enhances guidance for contributors.
238-244: API Server Setup:
The revised section for running the Cognee API server (with Docker compose instructions) aligns with the quickstart guide and improves the overall clarity of deployment steps.
250-253: Code of Conduct Accessibility:
The repositioned and reformatted Code of Conduct section enhances its visibility and clarity, making it easier for contributors to locate and understand.
Description
DCO Affirmation
I affirm that all code in every commit of this pull request conforms to the terms of the Topoteretes Developer Certificate of Origin
Summary by CodeRabbit