Page MenuHomePhabricator

[clang-tidy] adjust scripts to subsubsections in Release Notes
Needs ReviewPublic

Authored by Eugene.Zelenko on Fri, Jan 10, 11:29 AM.

Details

Summary

I added subsubsections for typical Clang-tidy entries in Release Notes, so now scripts are aware of this changes.

I don't have GitHub commit access, so please commit changes.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptFri, Jan 10, 11:29 AM
Herald added a subscriber: xazax.hun. · View Herald Transcript

LG with a nit, if this works.

clang-tools-extra/clang-tidy/add_new_check.py
223

"New check aliases" would be less ambiguous.

Renamed "New aliases" to "New check aliases".

Eugene.Zelenko marked an inline comment as done.Mon, Jan 13, 10:13 AM

Both scripts works fine. However rename script should also sort entries alphabetically, but probably this should be separate patch.

Both scripts works fine. However rename script should also sort entries alphabetically, but probably this should be separate patch.

Yep, sorting of the list on rename is a separate issue.
The patch LGTM.