feat(code-graph): bind property types in C#

Anthony Juckelrequested to merge
ajuckel/knowledge-graph:ajuckel/csharp-property-type into main

What does this MR do and why?

Adds a new binding to handle property declarations, storing the type information for the property. This fixes the skipped property test, and allows the graph to find the call to Point.DistanceTo via the Start property.

FYI, this is the first I'm looking around at this project. My interest was just piqued because I didn't realize GitLab was doing anything with Rust until I stumbled upon this. I didn't see a community fork, which has been the usual way that I've contributed to GitLab in the past, so I created my own private fork. If there's a different way this group would like to handle external contributions, I'm all ears.

Related Issues

I came upon the issue reading through #354 (closed).

Testing

Thus far, I've only tested by running the cargo tests locally. I'll inspect the pipeline after creating the MR to see if there are any follow-ups.

Performance Analysis

  • This merge request does not introduce any performance regression. If a performance regression is expected, explain why.

Merge request reports