Skip to content

feat: add hover docs for use-chain injected functions with @callback fallback#20

Open
aayushmau5 wants to merge 1 commit intoremoteoss:mainfrom
aayushmau5:ecto-hover-docs
Open

feat: add hover docs for use-chain injected functions with @callback fallback#20
aayushmau5 wants to merge 1 commit intoremoteoss:mainfrom
aayushmau5:ecto-hover-docs

Conversation

@aayushmau5
Copy link
Copy Markdown

FULL DISCLOSURE: This is all written by AI(through amp), but with manual steering where felt necessary. Please feel free to close this one if it doesn't match the expectation.

A PoC change that adds support for showing hover docs for use-chain injected function which holds their documentation in callbacks.

For example, Ecto's Repo use macro don't hold the documentation. They are actually present in the callbacks.

Screen.Recording.2026-04-13.at.12.03.41.AM.mov

@aayushmau5 aayushmau5 changed the title add hover docs for use-chain injected functions with @callback fallback feat: add hover docs for use-chain injected functions with @callback fallback Apr 13, 2026
@JesseHerrick
Copy link
Copy Markdown
Collaborator

Thanks for the contribution @aayushmau5, I haven't yet looked at the code, but this is a great contribution and I think a great idea! Will take a closer look tomorrow.

@JesseHerrick JesseHerrick self-requested a review April 13, 2026 03:03
@JesseHerrick
Copy link
Copy Markdown
Collaborator

@aayushmau5 I noticed that the PR is still in draft, is this intentional?

@aayushmau5 aayushmau5 marked this pull request as ready for review April 13, 2026 03:04
@aayushmau5
Copy link
Copy Markdown
Author

aayushmau5 commented Apr 13, 2026

Nope. Now up for review :)

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