Skip to content

Conversation

@capitan-davide
Copy link
Contributor

The check 'readability-braces-around-statements' do offer fixes!

@llvmbot
Copy link
Member

llvmbot commented Oct 9, 2025

@llvm/pr-subscribers-clang-tidy

@llvm/pr-subscribers-clang-tools-extra

Author: Davide Cunial (capitan-davide)

Changes

The check 'readability-braces-around-statements' do offer fixes!


Full diff: https://github.com/llvm/llvm-project/pull/162698.diff

1 Files Affected:

  • (modified) clang-tools-extra/docs/clang-tidy/checks/list.rst (+1-1)
diff --git a/clang-tools-extra/docs/clang-tidy/checks/list.rst b/clang-tools-extra/docs/clang-tidy/checks/list.rst index c490d2ece2e0a..7f7857ad39908 100644 --- a/clang-tools-extra/docs/clang-tidy/checks/list.rst +++ b/clang-tools-extra/docs/clang-tidy/checks/list.rst @@ -370,7 +370,7 @@ Clang-Tidy Checks :doc:`readability-avoid-nested-conditional-operator <readability/avoid-nested-conditional-operator>`, :doc:`readability-avoid-return-with-void-value <readability/avoid-return-with-void-value>`, "Yes" :doc:`readability-avoid-unconditional-preprocessor-if <readability/avoid-unconditional-preprocessor-if>`, - :doc:`readability-braces-around-statements <readability/braces-around-statements>`, + :doc:`readability-braces-around-statements <readability/braces-around-statements>`, "Yes" :doc:`readability-const-return-type <readability/const-return-type>`, "Yes" :doc:`readability-container-contains <readability/container-contains>`, "Yes" :doc:`readability-container-data-pointer <readability/container-data-pointer>`, "Yes" 
@localspook
Copy link
Contributor

This check has two aliases, neither of which is documented as providing fix-its. Presumably those should be updated as well?

:doc:`google-readability-braces-around-statements <google/readability-braces-around-statements>`, :doc:`readability-braces-around-statements <readability/braces-around-statements>`,

:doc:`hicpp-braces-around-statements <hicpp/braces-around-statements>`, :doc:`readability-braces-around-statements <readability/braces-around-statements>`,

@capitan-davide capitan-davide force-pushed the clang-tidy-update-list-rst branch from fac9b77 to 297349e Compare October 9, 2025 17:03
@capitan-davide
Copy link
Contributor Author

This check has two aliases, neither of which is documented as providing fix-its. Presumably those should be updated as well?

:doc:`google-readability-braces-around-statements <google/readability-braces-around-statements>`, :doc:`readability-braces-around-statements <readability/braces-around-statements>`,

:doc:`hicpp-braces-around-statements <hicpp/braces-around-statements>`, :doc:`readability-braces-around-statements <readability/braces-around-statements>`,

Amended

The check 'readability-braces-around-statements' do offer fixes!
@capitan-davide capitan-davide force-pushed the clang-tidy-update-list-rst branch from 297349e to c6416e4 Compare October 9, 2025 17:08
@vbvictor vbvictor changed the title [clang-tidy][NFC] Update list.rst [clang-tidy][NFC] Update list.rst fix-its info for readability-braces-around-statements check Oct 9, 2025
@vbvictor vbvictor merged commit 06cffb7 into llvm:main Oct 9, 2025
10 checks passed
@capitan-davide capitan-davide deleted the clang-tidy-update-list-rst branch October 10, 2025 07:42
DharuniRAcharya pushed a commit to DharuniRAcharya/llvm-project that referenced this pull request Oct 13, 2025
…-around-statements check (llvm#162698) The check 'readability-braces-around-statements' do offer fixes!
akadutta pushed a commit to akadutta/llvm-project that referenced this pull request Oct 14, 2025
…-around-statements check (llvm#162698) The check 'readability-braces-around-statements' do offer fixes!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment