axodox / AxoCover

Nice and free .Net code coverage support for Visual Studio with OpenCover.
https://marketplace.visualstudio.com/items?itemName=axodox1.AxoCover
MIT License
248 stars 60 forks source link

Changed .vsixmanifest to support Visual Studio 2019. #191

Open LoRdPMN opened 5 years ago

LoRdPMN commented 5 years ago

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 commented 5 years ago

CLA assistant check
All committers have signed the CLA.

ericop commented 5 years ago

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 commented 5 years ago

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

kjetilroe commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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

kjetilroe commented 5 years ago

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.

axodox commented 5 years ago

I will look into this over the weekend

axodox commented 5 years ago

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 commented 5 years ago

Sorry the new build is not good yet.

axodox commented 5 years ago

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 commented 5 years ago

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

MarkusAmshove commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

master-1.1.400 works fine on my

Jsparham777 commented 4 years ago

Hi all, whats the latest with this PR?

wickstargazer commented 3 years ago

Bump

rbaudrillard commented 2 years ago

Can it be done again for VS2022 ? Would be nice