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

[LiquidDoc] Show annotation completion suggestions when @ is typed #839

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

jamesmengo
Copy link
Contributor

@jamesmengo jamesmengo commented Mar 5, 2025

What are you adding in this PR?

Closes https://github.com/Shopify/developer-tools-team/issues/613

Renders completion suggestion for liquidDoc annotations while typing

Cursor - [Extension Development Host] mega-menu.liquid — horizon.mp4 (uploaded via Graphite)

What did you learn?

  • Didn't know about trigger characters before this but now I do!

Before you deploy

  • I included a minor bump changeset
  • My feature is backward compatible

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jamesmengo jamesmengo changed the title Register @ as a completions trigger character [LiquidDoc] Suggest Mar 5, 2025
@jamesmengo jamesmengo changed the title [LiquidDoc] Suggest [LiquidDoc] Show annotation completion suggestions Mar 5, 2025
@jamesmengo jamesmengo changed the title [LiquidDoc] Show annotation completion suggestions [LiquidDoc] Show annotation completion suggestions when @ is typed Mar 5, 2025
@jamesmengo jamesmengo force-pushed the jm/add_completions_trigger branch from f235c67 to 1dc0317 Compare March 5, 2025 01:01
@jamesmengo jamesmengo marked this pull request as ready for review March 5, 2025 01:02
@jamesmengo jamesmengo requested a review from a team as a code owner March 5, 2025 01:02
@jamesmengo jamesmengo requested review from aswamy and EvilGenius13 and removed request for a team March 5, 2025 01:02
@jamesmengo jamesmengo added the #gsd:44310 LiquidDoc label Mar 5, 2025 — with Graphite App
Copy link
Contributor

@EvilGenius13 EvilGenius13 left a comment

Choose a reason for hiding this comment

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

Works great during my 🎩 ! TIL about the triggers as well.

@jamesmengo jamesmengo merged commit f9c4c22 into main Mar 5, 2025
7 checks passed
@jamesmengo jamesmengo deleted the jm/add_completions_trigger branch March 5, 2025 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#gsd:44310 LiquidDoc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants