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

Remove minimum requirement for C# version 7 #26

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

Conversation

bao7uo
Copy link

@bao7uo bao7uo commented Jul 4, 2022

Adding explicit variable declarations for an out variable, instead of declaring them in the method call means that C# version 7 is not a minimum requirement, and different version of msbuild can be used. So this PR improves backward compatibility with earlier C# versions.

"Starting with C# 7.0, you can declare the out variable in the argument list of the method call, rather than in a separate variable declaration. " - https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/out-parameter-modifier

bao7uo added 2 commits July 4, 2022 21:40
This changes means that C# v7 is no longer a minimum requirement
Add explicity variable declaration for the out variable. This changes means that C# v7 is no longer a minimum requirement
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.

1 participant