Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add EvaluationTimeoutSample projects and readme file #263

Merged
merged 8 commits into from
Aug 2, 2022

Conversation

mpeyrotc
Copy link
Member

We are adding tutorials for Custom Managed Debugger Visualizers here and customers wanted to be able to download the projects for which we show code snippets. Hence, we are putting them here so that they have easy access to them.

@madskristensen
Copy link
Contributor

Is there a reason you're not using <PackageReference> but uses the old nuget.config files?

@mpeyrotc
Copy link
Member Author

mpeyrotc commented Aug 1, 2022

@madskristensen there is no reason, I assume it must have been default behavior from VS since they are .NET Framework projects.

@madskristensen
Copy link
Contributor

It would be nice to have these sample follow as many modern and best practices as possible. That includes using PackageReference. I recommend you change it, but it's not a strict rule and we can publish without it. Let me know what you want to do

@WardenGnaw
Copy link
Member

The WPF project cannot be SDK styled unless we upgrade the version of VS used to build the projects to be at least VS 2019.

image: Visual Studio 2017

Related: dotnet/installer#213

@mpeyrotc
Copy link
Member Author

mpeyrotc commented Aug 2, 2022

@madskristensen I think it is ready for check-in :)

@madskristensen madskristensen merged commit fa5e449 into microsoft:master Aug 2, 2022
@madskristensen
Copy link
Contributor

Thanks

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

Successfully merging this pull request may close these issues.

3 participants