-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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 Proposal]: Add GetDeclaredSymbol
overload for LocalFunctionStatementSyntax
#71028
Labels
api-approved
API was approved in API review, it can be implemented
Area-IDE
Concept-API
This issue involves adding, removing, clarification, or modification of an API.
Feature Request
Comments
CollinAlpert
added
Concept-API
This issue involves adding, removing, clarification, or modification of an API.
Feature Request
labels
Nov 30, 2023
dotnet-issue-labeler
bot
added
Area-Compilers
untriaged
Issues and PRs which have not yet been triaged by a lead
labels
Nov 30, 2023
Should likely be typed to return an IMethodSymbol |
That was my initial thought as well, however it seems the method in |
@CollinAlpert tweak here: #71034 |
Cool! I've tweaked the proposal. |
Awesome. Thanks. I'll drive this! |
CyrusNajmabadi
added
api-ready-for-review
API is ready for review, it is NOT ready for implementation
and removed
untriaged
Issues and PRs which have not yet been triaged by a lead
labels
Nov 30, 2023
API Review
Conclusion: Approved. |
333fred
added
api-approved
API was approved in API review, it can be implemented
and removed
api-ready-for-review
API is ready for review, it is NOT ready for implementation
labels
Dec 7, 2023
AlekseyTs
pushed a commit
that referenced
this issue
Dec 14, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
api-approved
API was approved in API review, it can be implemented
Area-IDE
Concept-API
This issue involves adding, removing, clarification, or modification of an API.
Feature Request
Background and Motivation
For the sake of completeness, an extension method on
SemanticModel
should exist forLocalFunctionStatementSyntax
. Note that such an overload already exists on theCSharpSemanticModel
.Proposed API
namespace Microsoft.CodeAnalysis.CSharp { public static class CSharpExtensions { + public static IMethodSymbol? GetDeclaredSymbol(this SemanticModel? semanticModel, LocalFunctionStatementSyntax node, CancellationToken cancellationToken = default(CancellationToken)); }
Usage Examples
Related to dotnet/roslyn-analyzers#7061
The text was updated successfully, but these errors were encountered: