Skip to content

checker: added support for hover function declaration#26789

Merged
JalonSolov merged 7 commits intovlang:masterfrom
cadamsdev:feat/hover-fn-declaration
Mar 31, 2026
Merged

checker: added support for hover function declaration#26789
JalonSolov merged 7 commits intovlang:masterfrom
cadamsdev:feat/hover-fn-declaration

Conversation

@cadamsdev
Copy link
Copy Markdown
Contributor

@cadamsdev cadamsdev commented Mar 29, 2026

This functionality allows users to see the function declaration when hovering over functions.

Preview

2026-03-29 15 15 09

Related PRs

@cadamsdev cadamsdev marked this pull request as ready for review March 29, 2026 16:59
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: 800777fe48

ℹ️ 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/fn.v
Comment thread vlib/v/checker/autocomplete.v Outdated
@StunxFS StunxFS changed the title feat(vls): added support for hover function declaration checker: added support for hover function declaration Mar 30, 2026
@JalonSolov
Copy link
Copy Markdown
Collaborator

Failing tests in vlib/v/tests/vls/autocomplete_module_test.v

@JalonSolov JalonSolov merged commit aa3390d into vlang:master Mar 31, 2026
89 of 90 checks passed
@cadamsdev cadamsdev deleted the feat/hover-fn-declaration branch March 31, 2026 14:36
medvednikov pushed a commit that referenced this pull request Apr 4, 2026
* feat: add support for hover fn declaration

* feat: implement hover functionality for function declarations

* fix: adjust column calculation for hover functionality in function declarations

* fix: enhance fn_decl to ensure correct file context for hover functionality

* fix: update function call autocomplete to include receiver type for hover details

* fix formatting

* fix tests
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