Skip to content

docs: drop hermes agent note from readme - #6

Merged
Syrup merged 1 commit into
mainfrom
docs/remove-hermes-note
Oct 13, 2026
Merged

Syrup merged 1 commit into
mainfrom
docs/remove-hermes-note

Conversation

@Syrup

@Syrup Syrup commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Removes the unbuilt Telegram integration row from the README's Not built yet table. The Hermes Agent reference was stale and explained nothing to readers.

Summary by CodeRabbit

  • Documentation
    • Removed Telegram integration from the list of planned features.

Copilot AI balanced review requested due to automatic review settings October 13, 2026 06:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b5f25a49-3657-409c-bef9-993c62cb47db
📥 Commits

Reviewing files that changed from the base of the PR and between 3a3a009 and fc94c1e.

📒 Files selected for processing (1)
  • README.md
💤 Files with no reviewable changes (1)
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The README’s “Not built yet” table no longer lists Telegram integration. The named-storage-profiles and per-chunk-data-keys entries remain.

Changes: README planned features

Layer / File(s) Summary
Update planned feature list
README.md
The Telegram integration entry was removed from the “Not built yet” table. The other listed entries remain.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to fc94c

The README update is ready to merge after normal checks.

Pre-merge checks · Passed 5
Passed Passed checks (5 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies a documentation change that removes the stale Hermes Agent note from the README. This matches the pull request objective.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

A rabbit read the list at dawn
One Telegram line was gone
The other plans stayed put
I nibbled clover by the foot
Then hopped along, content and fond

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

@Syrup
Syrup merged commit 9aa7334 into main Oct 13, 2026
5 checks passed
@Syrup
Syrup deleted the docs/remove-hermes-note branch October 13, 2026 07:01
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.

2 participants