-
Notifications
You must be signed in to change notification settings - Fork 36
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
"this" keyword required? #28
Comments
@jorgeyanesdiez Hi, the As for the namespace, I will think about it. Thank you for the suggestion! 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the following scenario, keyword
this
is required; I assume this is because you havetwo generic methods with both
TController : Controller
andTController : class
.This is annoying. Why do you need the
TController: class
version? Am I missing something?Additionally, it'd be great if your extension methods lived under another namespace, and not the official
Microsoft.AspNetCore.Mvc
, mainly for clarity and extensibility. For example, to fix the above I thought of making a wrapper class in my namespace so that only the methods withTController : Controller
are "locatable" in my code, but since I obviously need the officialMicrosoft.AspNetCore.Mvc
namespace, then I get method name clashes. What do you think?The text was updated successfully, but these errors were encountered: