Skip to content

Conversation

@hande-k
Copy link
Contributor

@hande-k hande-k commented Nov 6, 2025

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Other (please specify):

Screenshots/Videos (if applicable)

Pre-submission Checklist

  • I have tested my changes thoroughly before submitting this PR
  • This PR contains minimal changes necessary to address the issue/feature
  • My code follows the project's coding standards and style guidelines
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if applicable)
  • All new and existing tests pass
  • I have searched existing PRs to ensure this change hasn't been submitted already
  • I have linked any relevant issues in the description
  • My commits have clear and descriptive messages

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.

@pull-checklist
Copy link

pull-checklist bot commented Nov 6, 2025

Please make sure all the checkboxes are checked:

  • I have tested these changes locally.
  • I have reviewed the code changes.
  • I have added end-to-end and unit tests (if applicable).
  • I have updated the documentation and README.md file (if necessary).
  • I have removed unnecessary code and debug statements.
  • PR title is clear and follows the convention.
  • I have tagged reviewers or team members for feedback.

@hande-k hande-k requested a review from Vasilije1990 November 6, 2025 13:00
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 6, 2025

Walkthrough

This pull request updates the README's Demos & Examples section by renaming demo heading titles, updating demo link labels, and refreshing demo URLs. Specifically, it changes "Cognee Cloud Beta Demo" to "Persistent Agent Memory," updates associated demo links and URLs, renames "Simple GraphRAG Demo" to "Simple GraphRAG," and refreshes demo URLs for multiple demo entries.

Changes

Cohort / File(s) Summary
Demo section updates
README.md
Renamed "Cognee Cloud Beta Demo" heading to "Persistent Agent Memory" and updated demo link label and URL. Renamed "Simple GraphRAG Demo" to "Simple GraphRAG" and updated demo URL. Updated "Cognee with Ollama" demo URL.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • Vasilije1990
  • dexters1
  • borisarzentar

Poem

🐰 Demo links hop to new homes so fine,
Headings renamed in a clear design,
Persistent memories and graphs aligned—
Our README's path is now redesigned! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is completely empty of actual content—it only contains the template structure with no human-written Description, Type of Change selection, or reasoning provided. Add a human-written description explaining why videos were updated, which demos were changed, and complete the Type of Change checklist with your selections.
Title check ❓ Inconclusive The title 'chore: update videos' is partially related to the changeset but lacks specificity about which videos were updated or how. Consider revising the title to be more specific, such as 'chore: update demo video links in README' to better clarify the scope of changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-readme-videos

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1e53138 and a6db281.

📒 Files selected for processing (1)
  • README.md (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (15)
  • GitHub Check: CLI Tests / CLI Functionality Tests
  • GitHub Check: CLI Tests / CLI Integration Tests
  • GitHub Check: End-to-End Tests / Conversation sessions test
  • GitHub Check: End-to-End Tests / Test Feedback Enrichment
  • GitHub Check: End-to-End Tests / Test Entity Extraction
  • GitHub Check: End-to-End Tests / Test permissions with different situations in Cognee
  • GitHub Check: End-to-End Tests / Concurrent Subprocess access test
  • GitHub Check: End-to-End Tests / S3 Bucket Test
  • GitHub Check: Basic Tests / Run Simple Examples
  • GitHub Check: Basic Tests / Run Unit Tests
  • GitHub Check: Basic Tests / Run Integration Tests
  • GitHub Check: Basic Tests / Run Linting
  • GitHub Check: End-to-End Tests / Server Start Test
  • GitHub Check: End-to-End Tests / Deduplication Test
  • GitHub Check: End-to-End Tests / Run Telemetry Pipeline Test
🔇 Additional comments (1)
README.md (1)

183-194: Demo heading changes are isolated; no broken references found.

The verification confirms that the old heading names "Cognee Cloud Beta Demo" and "Simple GraphRAG Demo" are not referenced anywhere else in the codebase. The changes are isolated to the README.md demo section with correct markdown syntax. GitHub URL accessibility and content accuracy are best verified by the maintainers with access to the demo materials.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Vasilije1990 Vasilije1990 merged commit 26c18e9 into main Nov 6, 2025
122 of 125 checks passed
@Vasilije1990 Vasilije1990 deleted the update-readme-videos branch November 6, 2025 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants