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

"don't await on Task.FromResult" analyzer and code fix #10

Open
tugberkugurlu opened this issue Nov 17, 2014 · 2 comments
Open

"don't await on Task.FromResult" analyzer and code fix #10

tugberkugurlu opened this issue Nov 17, 2014 · 2 comments

Comments

@tugberkugurlu
Copy link

This is sad but yes, I have seen usages of this 😄 More info is available as a blog post. Also, have a look at AsyncPackage sample inside Roslyn code base to get an idea of how an async analyzer can be implemented.

@sharwell
Copy link
Member

The AsyncPackage sample is actually overly complicated for this case. You can just register a syntax analyzer for an AwaitExpression.

@tugberkugurlu
Copy link
Author

Yes, right.

BTW, do you know how that AsyncPackage will end up? They didn't ship it and I was thinking that it would be more meaningful to have all async specific diagnostics inside one package/VSIX and refactoring + diagnostics inside one VSIX.

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

No branches or pull requests

2 participants