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

Remove .vscode-test-web from extension package #292

Closed
kidonng opened this issue Aug 20, 2022 · 0 comments · Fixed by #293
Closed

Remove .vscode-test-web from extension package #292

kidonng opened this issue Aug 20, 2022 · 0 comments · Fixed by #293

Comments

@kidonng
Copy link
Contributor

kidonng commented Aug 20, 2022

I noticed the extension takes surprisingly large disk space:

$ du -sh ~/.vscode/extensions/wakatime.vscode-wakatime-*/{,.}* | sort -h | tail -n3
796K	/Users/kid/.vscode/extensions/wakatime.vscode-wakatime-19.0.1/dist
 53M	/Users/kid/.vscode/extensions/wakatime.vscode-wakatime-19.0.0/.vscode-test-web
 54M	total

It seems like .vscode-test-web is a dependency only used in development:

$ rg vscode-test-web ~/.vscode/extensions/wakatime.vscode-wakatime-*
/Users/kid/.vscode/extensions/wakatime.vscode-wakatime-19.0.0/package.json
53:		"open-in-browser": "vscode-test-web --open-devtools --version sources --extensionDevelopmentPath=. .",

Having it in the final package doesn't make sense. I suggest removing it from the package before publishing to the marketplace.

@kidonng kidonng changed the title Remove .vscode-test-web from extension bundle Remove .vscode-test-web from extension package Aug 20, 2022
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 a pull request may close this issue.

1 participant