Skip to content

docs: edited reactions in readme#173

Merged
highesttt merged 2 commits into
mainfrom
readme-reaction-update
Jun 4, 2026
Merged

docs: edited reactions in readme#173
highesttt merged 2 commits into
mainfrom
readme-reaction-update

Conversation

@highesttt

Copy link
Copy Markdown
Collaborator

No description provided.

@indent

indent Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor
PR Summary

Docs-only update to the README feature list to reflect that sending reactions is supported (limited to ~120 emojis), pointing readers to the implementation in pkg/connector/reaction.go. Replaces the previous "Receive ONLY" wording so the feature list matches actual connector behavior.

  • Updated the "Reaction support" bullet in README.md with the new send/receive description and a link to pkg/connector/reaction.go.

Issues

All clear! No issues remaining. 🎉

1 issue already resolved
  • README reaction bullet has inconsistent capitalization and reads awkwardly ("Receive Fully supported, send Limited to ~120 emojis"); consider rewording, e.g. "Reactions (receive: fully supported; send: limited to ~120 emojis — see reaction.go)". (fixed by commit 52433f1)

CI Checks

All CI checks passed on commit 52433f1.

@coderabbitai

coderabbitai Bot commented Jun 4, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 67315c44-e581-4a1e-a2de-52c5592b5e5e

📥 Commits

Reviewing files that changed from the base of the PR and between ba7f799 and 2b23022.

📒 Files selected for processing (1)
  • README.md
📜 Recent review details
⏰ 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). (5)
  • GitHub Check: Build 1.25
  • GitHub Check: Lint with 1.25
  • GitHub Check: build-docker
  • GitHub Check: Lint with 1.25
  • GitHub Check: build-docker
🔇 Additional comments (1)
README.md (1)

50-50: LGTM!


📝 Walkthrough

Summary by CodeRabbit

  • Documentation
    • Updated feature documentation to reflect enhanced reaction support capabilities, now enabling users to both receive and send reactions (send functionality limited to approximately 120 emojis).

Walkthrough

The PR updates the README feature list to document the reaction support capabilities more explicitly. The Reaction entry now specifies that receive is fully supported and send is limited to approximately 120 emojis, with a reference link to the reaction connector implementation.

Changes

Documentation Updates

Layer / File(s) Summary
Reaction support documentation
README.md
Feature list entry for reaction support is updated from "Receive ONLY" to specify "Receive fully supported" with "send limited to ~120 emojis" and includes a link to pkg/connector/reaction.go.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • beeper/matrix-line-messenger#172: Introduces or expands the reaction support implementation in pkg/connector/reaction.go that is now referenced in this README update.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided by the author, making it impossible to assess relevance to the changeset. Add a pull request description explaining the motivation and details of the reactions documentation update.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: updating the reactions documentation in the README file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch readme-reaction-update

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

Comment thread README.md Outdated
Co-authored-by: indent[bot] <216979840+indent[bot]@users.noreply.github.com>
@highesttt highesttt merged commit 2914bdb into main Jun 4, 2026
10 checks passed
@highesttt highesttt deleted the readme-reaction-update branch June 4, 2026 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant