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.

Sign up for GitHub

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

Jump to bottom

refactor: remove config/list if item newline format #1643

Merged

Peefy merged 1 commit into kcl-lang:main from Peefy:config-if-entry-format

Sep 11, 2024

Contributor

Peefy commented Sep 11, 2024

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

N
Y

refactor: remove config/list if item newline format


          refactor: remove config/list if item newline format

Signed-off-by: peefy <[email protected]>

He1pa approved these changes

View reviewed changes

Contributor

He1pa left a comment

LGTM

Collaborator

coveralls commented Sep 11, 2024

Pull Request Test Coverage Report for Build 10810569653

Details

4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
12 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 70.558%

Files with Coverage Reduction	New Missed Lines	%
kclvm/tools/src/LSP/src/state.rs	12	82.57%

Totals
Change from base Build 10792913935:	-0.01%
Covered Lines:	51217
Relevant Lines:	72588

💛 - Coveralls

Peefy merged commit 7344032 into kcl-lang:main

11 of 12 checks passed

Peefy deleted the config-if-entry-format branch

September 11, 2024 11:52

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet