|
1 | 1 | ### Reminder |
2 | | -- [ ] Have you linked an issue with this pull request? |
3 | | -- [ ] Have you added adequate unit tests and/or case tests for your pull request? |
4 | | -- [ ] Have you noticed possible changes of behavior below or in the linked issue? |
5 | | -- [ ] Have you explained the changes of codes in core modules of ESolver, HSolver, ElecState, Hamilt, Operator or Psi? (ignore if not applicable) |
| 2 | +- [ ] I have read `AGENTS.md` and `docs/developers_guide/agent_governance.md`. |
| 3 | +- [ ] I have linked an issue or explained why this PR does not need one. |
| 4 | +- [ ] I have added adequate unit tests and/or case tests, or explained why not. |
| 5 | +- [ ] I have listed the exact verification commands run and their results. |
| 6 | +- [ ] I have described user-visible behavior changes, including INPUT parameter changes. |
| 7 | +- [ ] I have explained core-module impact for ESolver, HSolver, ElecState, Hamilt, Operator, Psi, or other `source/` changes. |
| 8 | +- [ ] I have requested any needed governance exception below. |
6 | 9 |
|
7 | 10 | ### Linked Issue |
8 | 11 | Fix #... |
9 | 12 |
|
10 | 13 | ### Unit Tests and/or Case Tests for my changes |
11 | 14 | - A unit test is added for each new feature or bug fix. |
12 | 15 |
|
| 16 | +### Exact Verification Performed |
| 17 | +- Commands run: |
| 18 | +- Result summary: |
| 19 | +- Checks not run, with reason: |
| 20 | + |
13 | 21 | ### What's changed? |
14 | 22 | - Example: My changes might affect the performance of the application under certain conditions, and I have tested the impact on various scenarios... |
15 | 23 |
|
16 | | -### Any changes of core modules? (ignore if not applicable) |
17 | | -- Example: I have added a new virtual function in the esolver base class in order to ... |
| 24 | +### Governance Checklist |
| 25 | +- Global dependencies: no new `GlobalV`, `GlobalC`, or `PARAM` cross-layer control, or exception requested below. |
| 26 | +- Default parameters: no new default arguments added to existing interfaces, or exception requested below. |
| 27 | +- Headers: no unnecessary header dependencies or `.hpp` propagation, or rationale provided below. |
| 28 | +- Line endings: text files use LF; only `.bat` and `.cmd` use CRLF. |
| 29 | +- Build linkage: new source files are listed in the relevant `CMakeLists.txt`, or rationale provided below. |
| 30 | +- Documentation: behavior/interface changes include documentation updates, or no documentation update is required because ... |
| 31 | +- CodeRabbit: if automatic review has not started and the repository has CodeRabbit installed, request `@coderabbitai review`. |
| 32 | + |
| 33 | +### INPUT Parameter Changes |
| 34 | +- Parameters added/removed/changed: |
| 35 | +- `docs/parameters.yaml` updated: yes/no/not applicable |
| 36 | +- `docs/advanced/input_files/input-main.md` updated: yes/no/not applicable |
| 37 | +- If not updated, explain why no INPUT documentation update is required: |
| 38 | + |
| 39 | +### Core Module Impact |
| 40 | +- Affected core modules: |
| 41 | +- Risk summary: |
| 42 | +- Compatibility or performance impact: |
| 43 | + |
| 44 | +### Governance Exception |
| 45 | +- Rule: |
| 46 | +- Reason: |
| 47 | +- Scope: |
| 48 | +- User or maintenance risk: |
| 49 | +- Why the normal rule cannot be followed now: |
| 50 | +- Follow-up cleanup plan: |
| 51 | +- Requested approver: |
0 commit comments