-
Notifications
You must be signed in to change notification settings - Fork 41
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
VSIX ships a lot of unrelated files, including Roslyn #22
Comments
@Therzok ah yeah, something definitely went wrong here 🙂 trying to figure out how to optimize it. |
hm.. looks like most of it comes with "Microsoft.CodeAnalysis.EditorFeatures.Text" nuget package which is only needed for |
You can most likely ship the vsix without bundling Roslyn, but by having a dependyencu on the .net sdk extension |
The assemblies exist in the appdomain, and the host will redirect you to the version it uses. Basically, all the imported packagereference that already exist in VS have to not be locally copied. |
It should just reference dlls and allow the roslyn host to load them. Right now, the extension (along with some others) cause my VS to not show quick info. :D
The text was updated successfully, but these errors were encountered: