Skip to content

README.md: remove reference to Replier - #37

Open
nelsam wants to merge 1 commit into
go-language-server:mainfrom
nelsam:nelsam/replier-docs
Open

README.md: remove reference to Replier#37
nelsam wants to merge 1 commit into
go-language-server:mainfrom
nelsam:nelsam/replier-docs

Conversation

@nelsam

@nelsam nelsam commented Jul 7, 2026

Copy link
Copy Markdown

As of a685e84, the Replier no longer exists. Including it in the README is confusing. This updates the README to reference the modern Handler type.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the example handler signature in the README.md to return a result and an error directly, rather than using a Replier. The review feedback correctly notes that the surrounding explanatory text in the README should also be updated to reflect this change.

Comment thread README.md
As of a685e84, the Replier no longer
exists. Including it in the README is confusing. This updates the
README to reference the modern Handler type.
@nelsam
nelsam force-pushed the nelsam/replier-docs branch from c3334da to 42a328c Compare July 7, 2026 17:41
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