clang-tidy keeps nice release notes of what is added, have clang-format do the same.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
@Eugene.Zelenko I've noticed you are always giving excellent review feedback in clang-tidy especially around the documentation, I'd appreciate your eyes on finding the right level of documentation here.
Comment Actions
Build result: pass - 60143 tests passed, 0 failed and 729 were skipped.
Log files: console-log.txt, CMakeCache.txt
Comment Actions
General comments:
- Isn't Option(s) ... was/were added or Option(s) ... had been added are more correct from from point of view of English grammar? Same for older entries.
- It'll be great to nag developers to expand Release Notes when relevant code changes are made.
clang/docs/ReleaseNotes.rst | ||
---|---|---|
323 | and between Options and DeriveLineEnding are not necessary. | |
324 | I would suggest to use dot after newlines. | |
326 | UseCRLF should be enclosed in double back-ticks. | |
329 | to insert? |
and between Options and DeriveLineEnding are not necessary.