Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

feat(codeintel): Add precise auto-inferencing for dotnet#63348

Merged
mmanela merged 6 commits into
mainfrom
mmanela/graph-698-add-auto-inference-support-for-dotnet
Jun 21, 2024
Merged

feat(codeintel): Add precise auto-inferencing for dotnet#63348
mmanela merged 6 commits into
mainfrom
mmanela/graph-698-add-auto-inference-support-for-dotnet

Conversation

@mmanela

@mmanela mmanela commented Jun 19, 2024

Copy link
Copy Markdown
Contributor

Adds support for dotnet inference by adding the lua script and wiring it up in the inference engine.

Note: I am not adding support in this PR for nuget package inference.

Test plan

  • Updated unit tests
  • End to end manual validation

Changelog

  • Enable automatic precise indexing of .NET projects

Comment thread internal/codeintel/autoindexing/internal/inference/libs/indexes.go
Comment thread internal/codeintel/autoindexing/internal/inference/lua/dotnet.lua
Comment thread internal/codeintel/autoindexing/internal/inference/lua/dotnet.lua Outdated
Comment thread internal/codeintel/autoindexing/internal/inference/lua/BUILD.bazel Outdated
Comment thread internal/codeintel/autoindexing/internal/inference/lua/dotnet.lua Outdated
Comment thread internal/codeintel/autoindexing/internal/inference/lua/dotnet.lua
@varungandhi-src

Copy link
Copy Markdown
Contributor

Note: I am note adding support in this PR for nuget package inference.

How much work would it be to add this/what's the reason for skipping it right now? If it's a large task, can you file a Linear issue with a sketch of the solution?

@mmanela mmanela marked this pull request as ready for review June 20, 2024 13:05
@mmanela mmanela requested a review from varungandhi-src June 21, 2024 12:56
Comment thread internal/codeintel/autoindexing/internal/inference/lua/dotnet.lua
@mmanela mmanela merged commit cc9ecd9 into main Jun 21, 2024
@mmanela mmanela deleted the mmanela/graph-698-add-auto-inference-support-for-dotnet branch June 21, 2024 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants