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

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.

patware added 4 commits May 18, 2019 10:11
… 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