-
Notifications
You must be signed in to change notification settings - Fork 777
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
Support private module completions #16051
Conversation
3e62127
to
01772fd
Compare
Test this change out locally with the following install scripts (Action run 13466619999) VSCode
Azure CLI
|
Dotnet Test Results 78 files - 39 78 suites - 39 30m 42s ⏱️ - 15m 25s Results for commit 736e011. ± Comparison against base commit 6fcb976. This pull request removes 2181 and adds 1094 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
a447698
to
479ed50
Compare
af65f40
to
5ea8c28
Compare
bbaf353
to
8ae4324
Compare
d940471
to
704815e
Compare
657444f
to
fef0f0e
Compare
src/Bicep.Core/Registry/Catalog/Implementation/BaseModuleMetadataProvider.cs
Show resolved
Hide resolved
fc990ba
to
df0fd29
Compare
5f76ca3
to
41a8cac
Compare
src/Bicep.Core/Registry/Catalog/Implementation/PublicRegistries/PublicModuleMetadataProvider.cs
Outdated
Show resolved
Hide resolved
src/Bicep.Core/Registry/Catalog/Implementation/PublicRegistries/PublicModuleMetadataProvider.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Shenglong Li <[email protected]>
…nto sw/acr-completions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great. Thanks for implementing this!
Fixes #9038
Microsoft Reviewers: Open in CodeFlow