Skip to content

Support for clang tidy 19 #922

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Mar 19, 2025
Merged

Support for clang tidy 19 #922

merged 9 commits into from
Mar 19, 2025

Conversation

mgovers
Copy link
Member

@mgovers mgovers commented Mar 17, 2025

Make sure that the code compiles with the latest clang-tidy (Windows; shipped with latest Visual Studio)

We disable the following cfr. the offline discussion:

@mgovers mgovers added improvement Improvement on internal implementation do-not-merge This should not be merged labels Mar 17, 2025
@mgovers mgovers self-assigned this Mar 17, 2025
@mgovers mgovers marked this pull request as ready for review March 17, 2025 11:07
@mgovers mgovers marked this pull request as draft March 17, 2025 11:07
mgovers added 2 commits March 17, 2025 12:12
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>

revert readability-math-missing-parentheses and boost-use-ranges

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>

revert readability-redundant-member-init

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers force-pushed the feature/clang-tidy-19 branch from 467da0f to f7239d1 Compare March 17, 2025 11:19
mgovers added 3 commits March 17, 2025 12:37
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers marked this pull request as ready for review March 17, 2025 13:05
@mgovers mgovers removed the do-not-merge This should not be merged label Mar 17, 2025
@mgovers
Copy link
Member Author

mgovers commented Mar 17, 2025

@mgovers mgovers enabled auto-merge March 17, 2025 13:50
@mgovers mgovers changed the title Feature/clang tidy 19 Support for clang tidy 19 Mar 17, 2025
mgovers and others added 4 commits March 18, 2025 08:09
Signed-off-by: Martijn Govers <martijn.govers@alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
…ng-tidy-19

Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
@mgovers mgovers changed the base branch from main to fix/macos-bump March 19, 2025 09:02
@mgovers mgovers merged commit b2e27f5 into fix/macos-bump Mar 19, 2025
20 of 21 checks passed
@mgovers mgovers deleted the feature/clang-tidy-19 branch March 19, 2025 09:02
@mgovers mgovers restored the feature/clang-tidy-19 branch March 19, 2025 09:04
@mgovers mgovers mentioned this pull request Mar 19, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement on internal implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant