Repository navigation
Conversation
|
Some conflicts need to be solved. Thanks. @Growl1234 |
7073217 to
4d472ed
Compare
4d472ed to
5c7be9d
Compare
|
@ABACUS-PR-BOT Please take a final review on this PR. |
ABACUS-PR-BOT
left a comment
There was a problem hiding this comment.
Final static review of 324c2ef94d19: I found no new semantic issue in the 97 reformatted CMake files or the AddTest formatter metadata. Ordered command/argument comparisons and git diff --check passed; no configure/build/runtime or CI verification was performed.
The existing conflict discussion remains relevant: merging current develop (370ba685e670) produces conflicts in the LCAO DFT+U and PW DFT+U test CMakeLists. Preserve its dftu_base_io_write.cpp additions when resolving them.
Non-blocking: the PR description still contains the empty template. Please record the exact formatter/check commands and observed results, and explain why runtime tests and INPUT-doc updates are unnecessary for this formatting change.
Reminder
AGENTS.mdanddocs/developers_guide/agent_governance.md.source/changes.Linked Issue
Fix #
Unit Tests and/or Case Tests for my changes
What's changed?
Governance Notes