AxoCover icon indicating copy to clipboard operation
AxoCover copied to clipboard

Changed .vsixmanifest to support Visual Studio 2019.

Open LoRdPMN opened this issue 5 years ago • 20 comments

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.

LoRdPMN avatar Jan 25 '19 17:01 LoRdPMN

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jan 25 '19 17:01 CLAassistant

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!

ericop avatar Mar 05 '19 19:03 ericop

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

MarkusAmshove avatar Apr 03 '19 07:04 MarkusAmshove

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.

kjetilroe avatar Apr 03 '19 20:04 kjetilroe

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.

r-matsunaga avatar Apr 04 '19 06:04 r-matsunaga

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.

kjetilroe avatar Apr 04 '19 07:04 kjetilroe

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

MarkusAmshove avatar Apr 12 '19 07:04 MarkusAmshove

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.

kjetilroe avatar Apr 12 '19 14:04 kjetilroe

I will look into this over the weekend

axodox avatar Jul 10 '19 11:07 axodox

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 avatar Jul 15 '19 16:07 axodox

Sorry the new build is not good yet.

axodox avatar Jul 15 '19 16:07 axodox

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 avatar Jul 15 '19 17:07 axodox

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

axodox avatar Jul 15 '19 17:07 axodox

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 :-)

MarkusAmshove avatar Jul 16 '19 18:07 MarkusAmshove

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.

axodox avatar Jul 16 '19 19:07 axodox

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 !

SieurCartier avatar Aug 05 '19 16:08 SieurCartier

master-1.1.400 works fine on my

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

m-ringler avatar Sep 12 '19 12:09 m-ringler

Hi all, whats the latest with this PR?

Jsparham777 avatar Apr 19 '20 15:04 Jsparham777

Bump

wickstargazer avatar Nov 29 '20 11:11 wickstargazer

Can it be done again for VS2022 ? Would be nice

rbaudrillard avatar Jan 07 '22 22:01 rbaudrillard