Skip to content

checker,pref: implement hover method for vls#26821

Merged
JalonSolov merged 2 commits intovlang:masterfrom
dy-tea:vls-hover
Apr 10, 2026
Merged

checker,pref: implement hover method for vls#26821
JalonSolov merged 2 commits intovlang:masterfrom
dy-tea:vls-hover

Conversation

@dy-tea
Copy link
Copy Markdown
Member

@dy-tea dy-tea commented Apr 4, 2026

Current vls hover simply reuses completion info. This adds the hover method which can provide a bit more info.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 717d18be4b

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread vlib/v/checker/autocomplete.v Outdated
Comment thread vlib/v/pref/line_info.v
@JalonSolov JalonSolov closed this Apr 6, 2026
@JalonSolov JalonSolov reopened this Apr 6, 2026
@JalonSolov JalonSolov closed this Apr 9, 2026
@JalonSolov JalonSolov reopened this Apr 9, 2026
@JalonSolov JalonSolov merged commit 2f87177 into vlang:master Apr 10, 2026
227 of 279 checks passed
@dy-tea dy-tea deleted the vls-hover branch April 11, 2026 02:47
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