Skip to content
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

chore: clean up clang-tidy #3145

Merged
merged 6 commits into from
Apr 25, 2024
Merged

chore: clean up clang-tidy #3145

merged 6 commits into from
Apr 25, 2024

Conversation

AJPfleger
Copy link
Contributor

@AJPfleger AJPfleger commented Apr 25, 2024

  • Sort alphabetically
  • Add missing values to .clang-tidy
  • Reformat for readability and to keep track of changes

In a dream world, we have just a single place, where we can manage the checks.

@AJPfleger AJPfleger added the 🚧 WIP Work-in-progress label Apr 25, 2024
@AJPfleger AJPfleger added this to the next milestone Apr 25, 2024
@github-actions github-actions bot added the Infrastructure Changes to build tools, continous integration, ... label Apr 25, 2024
@AJPfleger AJPfleger changed the title ci: c-style static cast chore: clean up clang-tidy Apr 25, 2024
@AJPfleger AJPfleger removed the 🚧 WIP Work-in-progress label Apr 25, 2024
@andiwand
Copy link
Contributor

I think clang tidy would actually discover .clang-tidy automatically but there was a reason to keep them separate which I cannot remember. maybe @paulgessinger ?

@kodiakhq kodiakhq bot merged commit 8821e7b into acts-project:main Apr 25, 2024
57 checks passed
@AJPfleger AJPfleger deleted the cast-away branch April 25, 2024 14:56
@paulgessinger
Copy link
Member

My original thinking was that we could steer which warnings to enable based on CMake logic. Maybe we don't need that though.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reason for his extra file is that there's a way (that we don't currently use) is to set a non-zero threshold of tolerable warnings.

@acts-project-service acts-project-service added the Fails Athena tests This PR causes a failure in the Athena tests label Apr 25, 2024
EleniXoch pushed a commit to EleniXoch/acts that referenced this pull request May 6, 2024
- Sort alphabetically
- Add missing values to `.clang-tidy`
- Reformat for readability and to keep track of changes

In a dream world, we have just a single place, where we can manage the checks.
@andiwand andiwand modified the milestones: next, v35.0.0 May 17, 2024
asalzburger pushed a commit to asalzburger/acts that referenced this pull request May 21, 2024
- Sort alphabetically
- Add missing values to `.clang-tidy`
- Reformat for readability and to keep track of changes

In a dream world, we have just a single place, where we can manage the checks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fails Athena tests This PR causes a failure in the Athena tests Infrastructure Changes to build tools, continous integration, ...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants