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

docs: generate vscode snippets #13

Closed
wants to merge 4 commits into from
Closed

Conversation

hyoban
Copy link
Contributor

@hyoban hyoban commented May 7, 2024

Description

Linked Issues

Additional context

I'm not sure if the format of the command follows specific rules. So I only generate for the full command name and @ alias

Copy link

netlify bot commented May 7, 2024

Deploy Preview for resonant-bonbon-a310e6 ready!

Name Link
🔨 Latest commit 2c646ba
🔍 Latest deploy log https://app.netlify.com/sites/resonant-bonbon-a310e6/deploys/663a34850c6de30008f7d85d
😎 Deploy Preview https://deploy-preview-13--resonant-bonbon-a310e6.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@antfu
Copy link
Owner

antfu commented May 7, 2024

I am not sure if typing epc would be easier than /// 😅

@hyoban
Copy link
Contributor Author

hyoban commented May 7, 2024

Oh, yes. I agree with this. I will modify it later. You can also modify it directly now. I don't have a computer with me now.

@antfu
Copy link
Owner

antfu commented May 7, 2024

In that sense - do we still need the snippet? I guess with command shorthand they are already short enough to type

@hyoban
Copy link
Contributor Author

hyoban commented May 8, 2024

OK, closing this PR for now. Since I don’t like abbreviations very much, it’s nice to be able to autocomplete the commands.

@hyoban hyoban closed this May 8, 2024
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