kcl-lang / kcl-go

KCL Go SDK
https://pkg.go.dev/kcl-lang.io/kcl-go@main
Apache License 2.0
52 stars 26 forks source link

Add all `service.KclvmService` methods to `NativeServiceClient` #252

Closed bozaro closed 8 months ago

bozaro commented 8 months 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):

pkg/native/client.go

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

Add all service.KclvmService methods to NativeServiceClient.

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:

github-actions[bot] commented 8 months ago

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

coveralls commented 8 months ago

Pull Request Test Coverage Report for Build 8231441790

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/native/client.go 11 65 16.92%
<!-- Total: 11 65 16.92% -->
Files with Coverage Reduction New Missed Lines %
pkg/native/client.go 3 38.53%
<!-- Total: 3 -->
Totals Coverage Status
Change from base Build 8227969953: -0.6%
Covered Lines: 3126
Relevant Lines: 5028

💛 - Coveralls
Peefy commented 8 months ago

Hello @bozaro Please sign the DCO and CLA. Thank you for the contribution. Code LGTM!

bozaro commented 8 months ago

I have read the CLA Document and I hereby sign the CLA