-
Notifications
You must be signed in to change notification settings - Fork 4k
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
API Request: TypeSymbolExtensions #64
Comments
I could add them to RoslynExts, but it requires other internal class. @OmerRaviv these extension are not on |
Stackoverflow user "Mac" might want to use methods from a similar API, INamedTypeSymbolExtensions. http://stackoverflow.com/questions/28234052/finding-all-class-declarations-than-inherit-from-another-with-roslyn @AnthonyDGreen Should this be a separate issue, or will we be evaluating a larger set of *Extensions classes as part of this API review? |
Thanks David, I've updated the post with the solution I hacked out, not sure if it is using bad practices though so if there is a ready made internal api for this it would be useful. |
any progress on this? |
Apparently Microsoft's other project (Roslyn Analyzers) also duplicates some of these extensions. |
A request for this should go through our "api proposal" path. Note that these APIs are unlikely to be exposed as they are really 'feature-level' apis that encode feature semantics and change as necessary. As a public API we would be unable to do that as it would likely break people (or every change would need some way to make it opt-in only, which would greatly increase complexity). |
Forking this feature request from #16
Original post: #16 (comment) by @MrJul
Pertinent APIs: http://source.roslyn.codeplex.com/#Microsoft.CodeAnalysis.CSharp/Symbols/TypeSymbolExtensions.cs,8f632696b9da5eba
In my team, I've seen different developers copy-paste portions of ITypeSymbolExtensions into our own project (e.g.
IsNullableType
,GetNullableUnderlyingType
,GetBaseTypes
), which is a shame - these methods are a convenience I'm sure many other people will find useful as well.The text was updated successfully, but these errors were encountered: