Skip to content

Detect enclosing function for C##885

Draft
xaldama wants to merge 1 commit into
xabier.aldama/detect_method_typescriptfrom
xabier.aldama/detect_method_csharp
Draft

Detect enclosing function for C##885
xaldama wants to merge 1 commit into
xabier.aldama/detect_method_typescriptfrom
xabier.aldama/detect_method_csharp

Conversation

@xaldama

@xaldama xaldama commented Apr 20, 2026

Copy link
Copy Markdown

Summary

  • Implements enclosing function detection for C# (FQN format: Namespace.ClassName.MethodName(ParamType1, ParamType2))
  • Handles methods, constructors, and local functions
  • Supports nested namespaces and the C# 10 file-scoped namespace Foo.Bar; syntax

Test plan

  • Unit tests in csharp/methods.rs cover methods, constructors, local functions, namespaces, parameters, and attribute decorators

@xaldama xaldama requested a review from a team as a code owner April 20, 2026 13:44
@xaldama xaldama requested review from colemaring and removed request for a team April 20, 2026 13:44
@xaldama xaldama marked this pull request as draft April 20, 2026 13:44
@datadog-datadog-prod-us1

datadog-datadog-prod-us1 Bot commented Apr 20, 2026

Copy link
Copy Markdown

🎯 Code Coverage (details)
Patch Coverage: 91.94%
Overall Coverage: 85.17% (+0.05%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 2ee398a | Docs | Datadog PR Page | Give us feedback!

@xaldama xaldama force-pushed the xabier.aldama/detect_method_typescript branch from 7ec57a1 to ad76f59 Compare April 20, 2026 14:07
@xaldama xaldama force-pushed the xabier.aldama/detect_method_csharp branch from c0cb80b to 2ee398a Compare April 20, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant