kcl-lang / kcl

KCL Programming Language (CNCF Sandbox Project). https://kcl-lang.io
https://kcl-lang.io
Apache License 2.0
1.7k stars 119 forks source link

feat: inlayhint for config key type #1659

Closed He1pa closed 1 month ago

He1pa commented 1 month ago

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

image

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

Peefy commented 1 month ago

I think the discussion is about an auto-completion feature rather than a type hint feature. 🤔

coveralls commented 1 month ago

Pull Request Test Coverage Report for Build 10990778156

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
kclvm/sema/src/ty/mod.rs 14 15 93.33%
kclvm/sema/src/advanced_resolver/node.rs 29 32 90.63%
<!-- Total: 43 47 91.49% -->
Files with Coverage Reduction New Missed Lines %
kclvm/tools/src/LSP/src/tests.rs 1 98.61%
kclvm/tools/src/LSP/src/state.rs 12 82.73%
<!-- Total: 13 -->
Totals Coverage Status
Change from base Build 10917873891: -0.002%
Covered Lines: 51306
Relevant Lines: 72681

💛 - Coveralls