fix lint check script - Githubissues

KusionStack / konfig

Shared repository of application models and components, and CI suite for GitOps workflows

Apache License 2.0

27 stars 32 forks source link

fix lint check script #50

Closed amyXia1994 closed 2 years ago

amyXia1994 commented 2 years ago

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

[X] N
[ ] Y

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

the lint check script is broken after merging PR #16 , this PR fixes it to the full.

hack/lint_check.py

here's an example of the effect of lint check after this fix, and it works fine now: https://github.com/KusionStack/konfig/actions/runs/3278253763/jobs/5396444153

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

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

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

[ ] N
[ ] Y

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

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

None