Skip to content
This repository has been archived by the owner on Sep 20, 2021. It is now read-only.

Updated the VSIX to make it Visual Studio 2019 compatible #72

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

patware
Copy link

@patware patware commented May 18, 2019

Many firsts for me:

  • First "real" use of git
  • First time git contribution (fork+pull request)
  • First Visual Studio Extension

A good code review will be necessary, lol.

I took the liberty of changing the template of the generated WPF451 project type, to reflect new changes in the CommonServiceLocator namespace, and small changes to make the generated code compile without warning/error.

… WPF451 project template. Many things were bothering me:

ViewModelLocator:
- using CommonServiceLocator;

.vstemplate:
- now points to the up-to-date NuGet versions of CommonServiceLocator and MvvmLightLibs

.csproj:
- valid ToolsVersion
- RunCodeAnalysis is now set to false: CA0507 (https://docs.microsoft.com/en-us/visualstudio/code-quality/fxcop-analyzers-faq?view=vs-2019#i-get-warning-ca0507-even-after-ive-installed-the-fxcop-analyzers-nuget-package)

No warnings or code change
@simonbuehler
Copy link

any reason why this isn't geting merged? its nearly a year now!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants