Skip to content
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

feature: kcl lint needs a feature like go build ./... that checks all of the KCL code, not just the compiled entry code #1691

Open
zong-zhe opened this issue Oct 14, 2024 · 2 comments · May be fixed by #1854
Assignees
Labels
enhancement New feature or request

Comments

@zong-zhe
Copy link
Contributor

Feature Request

Is your feature request related to a problem? Please describe:

I would like kcl lint to have a feature that checks the all KCL codes instead of just the code associated with the compiled entry.

Describe the feature you'd like:

Looks like go build ./... or cargo fmt --all, this feature will iterate over all packages, not just the compile entry.

Describe alternatives you've considered:

go build ./... or cargo fmt --all

Teachability, Documentation, Adoption, Migration Strategy:

@zong-zhe zong-zhe added the enhancement New feature or request label Oct 14, 2024
@zong-zhe zong-zhe added this to the v0.11.0 Release milestone Oct 14, 2024
@Siddhi-agg
Copy link
Contributor

I would like to work on this. Can you please point me towards the relevant entry code of the kcl lint command?

CC: @He1pa @Peefy @zong-zhe

@He1pa
Copy link
Contributor

He1pa commented Feb 10, 2025

I would like to work on this. Can you please point me towards the relevant entry code of the kcl lint command?

CC: @He1pa @Peefy @zong-zhe

https://github.com/kcl-lang/kcl/blob/main/kclvm/tools/src/lint/mod.rs

@Siddhi-agg Siddhi-agg linked a pull request Feb 11, 2025 that will close this issue
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

3 participants