Skip to content

Conversation

@4m4n-x-B4w4ne
Copy link
Contributor

@4m4n-x-B4w4ne 4m4n-x-B4w4ne commented Jan 6, 2025

This Pull Request is for fixing the failed build-bot checks for the PR(#120087) which references issue (#36323)

Added new options in ImplicitBoolConversionCheck CheckConversionToBool and CheckConversionFromBool.
Added CheckConversionToBool and CheckConversionFromBool Options in the header
Added new test to check the new options added in the ImplicitBoolConversionCheck.cpp
Added new Options in ImplicitBoolConversionCheck.
Improved Preview By adding blank line.
Formatted using clang-format.
Documented the new options.
Rewritten the RUN directives.
Fixed by separating options by ',' .
Fixed Bugs regarding wrong variable names.
Fixed Bugs regarding mismatched named variables.
Improved Readibility and Description.
Fixed Bugs related to check-suffix.
Just for testing purpose of BuildKite.
Wrote basic level tests. (Similar to the tests in other check files).
Updated for all permutations of configurations options and also for  the more comprehensive tests.
FIxed Clang formatting issue.
@github-actions
Copy link

github-actions bot commented Jan 6, 2025

⚠️ We detected that you are using a GitHub private e-mail address to contribute to the repo.
Please turn off Keep my email addresses private setting in your account.
See LLVM Discourse for more information.

@Sh0g0-1758 Sh0g0-1758 requested review from 5chmidti, HerrCai0907 and PiotrZSL and removed request for 5chmidti, HerrCai0907 and PiotrZSL January 8, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant