search

axodox / AxoCover

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

extension prompts to enable telemetry after every visual studio update #160

Open ZigMeowNyan opened 6 years ago

ZigMeowNyan commented 6 years ago

If telemetry hasn't been enabled in VS 2017, AxoCover will prompt to enable it after every VS update. If there's going to be a nag "feature" it should be configurable.

axodox commented 6 years ago

The nag is not a feature, this code was made before VS 2017 was even released and before that updates were quite rare so there was no consideration. I think this is caused by that when VS is updated the settings of AxoCover are lost, I will need to look into save them in a way that the update will not erase it.

axodox commented 6 years ago

BTW it even prompts when it is enabled.

ZigMeowNyan commented 6 years ago

Ah, I didn't know. Thank you for enlightening me. Do you think there's a VS 2017-specific way to save configuration settings? I did some quick googling, but I'm not seeing any hints toward that.

axodox commented 6 years ago

I used the built-in settings capability of .Net, and while it works fine usually, it seems it does not tolerate reinstalls without forgetting all the settings. I guess I was not annoyed by this so much so far because at work I am using VS 2015 and that is not getting much updates. I might have some time to check it this weekend.

axodox commented 6 years ago

So the problem is likely not VS 2017 itself, but rather that there are updates coming every other week or so, which trigger this loss of settings more often.