Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include template arguments for operation in intellisense #3183

Merged
merged 13 commits into from
Apr 18, 2024

Conversation

RodgeFu
Copy link
Contributor

@RodgeFu RodgeFu commented Apr 17, 2024

fix #2423

Include template arguments for operation in intellisense

@RodgeFu RodgeFu changed the title Fix/include template args Include template arguments for operation and interface in intellisense Apr 17, 2024
@RodgeFu RodgeFu changed the title Include template arguments for operation and interface in intellisense Include template arguments for operation in intellisense Apr 17, 2024
@timotheeguerin
Copy link
Member

/azp run typespec - PR Tools

@microsoft microsoft deleted a comment from azure-pipelines bot Apr 17, 2024
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-sdk
Copy link
Collaborator

azure-sdk commented Apr 17, 2024

All changed packages have been documented.

  • @typespec/compiler
Show changes

@typespec/compiler - feature ✏️

Show template parameters when hovering on an operation template

@azure-sdk
Copy link
Collaborator

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3183/

Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3183/

Copy link
Member

@timotheeguerin timotheeguerin left a comment

Choose a reason for hiding this comment

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

Looks nice
image

just a few changes

…2.md

Co-authored-by: Timothee Guerin <timothee.guerin@outlook.com>
Copy link
Member

@timotheeguerin timotheeguerin left a comment

Choose a reason for hiding this comment

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

Awesome this looks great!

@timotheeguerin timotheeguerin merged commit 9fcc90f into microsoft:main Apr 18, 2024
21 checks passed
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.

Intelisense should show a template signature
3 participants