-
-
Notifications
You must be signed in to change notification settings - Fork 118
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
ns2.1 and TryResolveStateMachineMethod #125
ns2.1 and TryResolveStateMachineMethod #125
Conversation
Is lang |
It "works on my machine" as it builds here. I see CI has it:
|
I'll remove it |
Oddly enough only failed on Ubuntu |
I'm going to check my casing in another project I have that was having the same issue with nullability 😅 |
Also a test to show its doing something? (Resolving the Iterator) |
…hub.com:bruno-garcia/Ben.Demystifier into feat/ns2.1-runtime-TryResolveStateMachineMethod
<PropertyGroup> | ||
<LangVersion>latest</LangVersion> | ||
</PropertyGroup> | ||
<PropertyGroup Condition="'$(TargetFramework)' != 'netstandard2.0' and !$(TargetFramework.StartsWith('netcoreapp2')) and !$(TargetFramework.StartsWith('net4'))"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adding net6
or any other tfm in the future would include HAS_ASYNC_ENUMERATOR
without anyone having to remember opting-in.
Thank you :) |
it's an easy change to discuss while looking at the change.
AsyncIteratorStateMachineAttribute
Nice CI is there! 🚀