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

Changed .vsixmanifest to support Visual Studio 2019. #191

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

Conversation

LoRdPMN
Copy link

@LoRdPMN LoRdPMN commented Jan 25, 2019

I really love this extension, so I am trying to contribute with #188 , following this article: https://blogs.msdn.microsoft.com/visualstudio/2018/09/26/how-to-upgrade-extensions-to-support-visual-studio-2019/

As said there, only changing the .vsixmanifest may work things out. I couldn't test the extension locally because I failed to setup the environment to debug, so if someone could run it prior to accepting, would be greatly appreciated!

Sorry if I did something wrong in regards of the pull request, I'm unexperienced with GitHub.

@CLAassistant
Copy link

CLAassistant commented Jan 25, 2019

CLA assistant check
All committers have signed the CLA.

@ericop
Copy link

ericop commented Mar 5, 2019

Yeah if someone can solve this before April when VS2019 goes live I know it'd help my team. We are all about the OpenCover and we all like what this extension brings to Visual Studio!

@MarkusAmshove
Copy link

Gave you tested this with the released 2019? It complains about missing CoreEditor for me

@kjetilroe
Copy link

Even with this manifest, it does not seem to identify VS 2019. Something more is needed for this extension to work. I have tried to do some investigation, but can't seem to find out what is preventing the extension to identify VS 2019 as a valid installation target.

@r-matsunaga
Copy link

VsixUpdater(https://github.com/axodox/VsixUpdater) works to override these tags when make .vsix file.
when I remove this nuget package and build, I could install and run in VS2019.
...perhaps this is true way that fix VsixUpdator to support VS2019.

@kjetilroe
Copy link

r-matsunaga you are right :-) I also managed to make it install on VS 2019 after removing the VsixUpdater. It looks like that module is overriding what is entered in the manifest file, and forcing the module to not identify VS 2019 as a valid target. Thank you very much for the tip.

@MarkusAmshove
Copy link

Is there a released VSIX somewhere? I can't build AxoCover locally :/

@kjetilroe
Copy link

I have uploaded my compiled vsix to dropbox: https://www.dropbox.com/s/1s8oodc4rm3f8fc/AxoCover.vsix?dl=0

It seems to be working fine. Only issue is the yellow bar on the top upon startup, saying that it will not be compatible with future versions of VS.

Copy link

@kjetilroe kjetilroe left a comment

Choose a reason for hiding this comment

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

VsixUpdater also need to be removed before it will run on VS 2019.

aelking
aelking approved these changes Jul 9, 2019
Copy link

@aelking aelking left a comment

Choose a reason for hiding this comment

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

Is this going to be approved anytime soon, VS2019 has been out for a while, and this is something that is really missing.

@axodox
Copy link
Owner

axodox commented Jul 10, 2019

I will look into this over the weekend

@axodox
Copy link
Owner

axodox commented Jul 15, 2019

I have created VSIX updater to maintain VS2012 compatibility (still used at my company by some people pn legacy projects) and to decrease VSIX size. I have updated the VS version there. Nuget is still doing validation, so I will have to wait some time to rebuild AxoCover.

@axodox
Copy link
Owner

axodox commented Jul 15, 2019

Sorry the new build is not good yet.

@axodox
Copy link
Owner

axodox commented Jul 15, 2019

I checked again it seems that the original reason for VSIX updater - that it was not possible to build packages which are compatible from VS2012 to VS2017 is not a problem anymore, but I would still keep it for its compression ability. I have updated it, and the VSIX manifest, so now new builds should work on VS2019. For the async loading further work is required.

@axodox
Copy link
Owner

axodox commented Jul 15, 2019

Please check this build on VS 2019 and 2012 (and between :D): https://github.com/axodox/AxoCover/releases/tag/master-1.1.400

@MarkusAmshove
Copy link

The build works fine on VS 2019, I even don't get the yellow warning about deprecated plugin stuff that came up with the other vsix :-)

@axodox
Copy link
Owner

axodox commented Jul 16, 2019

I will wait a bit for more confirmations, and possible error reports. Until that you may use the above build. If everything is fine I will push this out to VS marketplace in a few weeks.

@SieurCartier
Copy link

SieurCartier commented Aug 5, 2019

Hi @axodox ,

Tested the previously mentionned build on VS 19 (Version 16.1.16) and it works fine, except warning from VS saying : "Visual Studio did not load one or more extensions that were using deprecated APIs" since I added the extension ...

Great work none the less, really appreciated !

@m-ringler
Copy link

master-1.1.400 works fine on my

  • VS2019 Professional v16.2.5 (with warning about deprecated APIs)
  • VS2017 Professional v15.9.12

@hieuxlu hieuxlu mentioned this pull request Oct 16, 2019
@Jsparham777
Copy link

Hi all, whats the latest with this PR?

@wickstargazer
Copy link

Bump

@rbaudrillard
Copy link

rbaudrillard commented Jan 7, 2022

Can it be done again for VS2022 ?
Would be nice

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.