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

chore: integrate minor .vscode/ tweaks #2857

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Repiteo
Copy link
Contributor

@Repiteo Repiteo commented Jan 26, 2025

Description

Addresses a few straggler issues in the .vscode/ files:

  • pwa-extensionHost replaced with extensionHost, as the former was an experimental monicker that's no longer required. EDIT: Still the suggested syntax (intellisense error?)
  • jock.svg set as svg formatter, because otherwise we're just recommending that extension for no real reason.
  • For convenience, added default testing task.

Contribution Guidelines

@@ -15,7 +15,7 @@
},
{
"name": "Run Web Extension in VS Code",
"type": "pwa-extensionHost",
"type": "extensionHost",
Copy link
Member

Choose a reason for hiding this comment

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

I was wondering if you can show a link if it's not required anymore? In their documentation I can still find a reference where they are using it. That's why I'm a little bit confused now:
https://code.visualstudio.com/api/extension-guides/web-extensions#test-your-web-extension-in-vs-code-running-on-desktop

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Huh! I was tipped off by intellisense saying it was invalid, but that's indeed the syntax shown in the example. Gonna revert that change; if I find anything concrete, I'll propose it in a separate PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For posterity:
25-01-28 10-06-16 Code

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