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.

Sign up for GitHub

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

Jump to bottom

AG0018 - Change to not error on Controller #137

Open

dicko2 opened this issue Oct 27, 2019 · 1 comment

Contributor

dicko2 commented Oct 27, 2019

If we use this rule on Ctrl then all methods need to be appended with "Async" doesn't make sense

yrshaikh mentioned this issue

AG0018 - Change not to error on ctrl #142

Closed

Member

szaboopeeter commented Jul 23, 2020 •

edited

Loading

There is this not-so-much advertised feature since 3.0, which makes makes me think that we might wanna just add Async suffix for async actions: https://docs.microsoft.com/en-us/aspnet/core/migration/22-to-30?view=aspnetcore-3.1&tabs=visual-studio#async-suffix-removal-from-controller-action-names

Edit: though there seems to be some issues with that, like this: dotnet/aspnetcore#14103

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