|
| 1 | +<!-- Thank you for contributing to KusionStack! |
| 2 | +
|
| 3 | +Note: |
| 4 | +
|
| 5 | +1. With pull requests: |
| 6 | +
|
| 7 | + - Open your pull request against "main" |
| 8 | + - Your pull request should have no more than two commits, if not you should squash them. |
| 9 | + - It should pass all tests in the available continuous integration systems such as GitHub Actions. |
| 10 | + - You should add/modify tests to cover your proposed code changes. |
| 11 | + - If your pull request contains a new feature, please document it on the README. |
| 12 | +
|
| 13 | +2. Please create an issue first to describe the problem. |
| 14 | +
|
| 15 | + We recommend that link the issue with the PR in the following question. |
| 16 | + For more info, check https://kusionstack.io/docs/governance/contribute/ |
| 17 | +--> |
| 18 | + |
| 19 | +#### 1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".): |
| 20 | + |
| 21 | +- [ ] N |
| 22 | +- [ ] Y |
| 23 | + |
| 24 | +<!-- You can add issue references here. |
| 25 | + e.g. |
| 26 | + fix #123, re #123, |
| 27 | + fix https://github.com/XXX/issues/44 |
| 28 | +--> |
| 29 | + |
| 30 | +#### 2. What is the scope of this PR (e.g. component or file name): |
| 31 | + |
| 32 | +<!-- You can add the scope of this change here. |
| 33 | + e.g. |
| 34 | + /src/server/core.rs, |
| 35 | + kusionstack/KCLVM/kclvm-parser |
| 36 | +--> |
| 37 | + |
| 38 | +#### 3. Provide a description of the PR(e.g. more details, effects, motivations or doc link): |
| 39 | + |
| 40 | +<!-- You can choose a brief description here --> |
| 41 | +- [ ] Affects user behaviors |
| 42 | +- [ ] Contains syntax changes |
| 43 | +- [ ] Contains variable changes |
| 44 | +- [ ] Contains experimental features |
| 45 | +- [ ] Performance regression: Consumes more CPU |
| 46 | +- [ ] Performance regression: Consumes more Memory |
| 47 | +- [ ] Other |
| 48 | + |
| 49 | +<!-- You can add more details here. |
| 50 | + e.g. |
| 51 | + Call method "XXXX" to ..... in order to ...., |
| 52 | + More details: https://XXXX.com/doc...... |
| 53 | +--> |
| 54 | + |
| 55 | +#### 4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link): |
| 56 | + |
| 57 | +- [ ] N |
| 58 | +- [ ] Y |
| 59 | + |
| 60 | +<!-- You can add more details here. |
| 61 | + e.g. |
| 62 | + Calling method "XXXX" will cause the "XXXX", "XXXX" modules to be affected. |
| 63 | + More details: https://XXXX.com/doc...... |
| 64 | +--> |
| 65 | + |
| 66 | +#### 5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links: |
| 67 | + |
| 68 | +<!-- You can choose a brief description here --> |
| 69 | +- [ ] Unit test |
| 70 | +- [ ] Integration test |
| 71 | +- [ ] Benchmark (add benchmark stats below) |
| 72 | +- [ ] Manual test (add detailed scripts or steps below) |
| 73 | +- [ ] Other |
| 74 | + |
| 75 | +<!-- You can add more details here. |
| 76 | +e.g. |
| 77 | +The test case in XXXX is used to ..... |
| 78 | +test cases in /src/tests/XXXXX |
| 79 | +test cases https://github.com/XXX/pull/44 |
| 80 | +benchmark stats: time XXX ms |
| 81 | +--> |
| 82 | + |
| 83 | +#### 6. Release note |
| 84 | + |
| 85 | +<!-- compatibility change, improvement, bugfix, and new feature need a release note --> |
| 86 | + |
| 87 | +Please refer to [Release Notes Language Style Guide](https://kusionstack.io/docs/governance/release-policy/) to write a quality release note. |
| 88 | + |
| 89 | +```release-note |
| 90 | +None |
| 91 | +``` |
0 commit comments