-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
Throw out some inline
s
#64110
Throw out some inline
s
#64110
Conversation
-readability-redundant-inline-specifier
readability-redundant-inline-specifier
This is an automated comment for commit 4d642a0 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page
Successful checks
|
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
readability-redundant-inline-specifier
inline
s
Tests, including perf tests LGTM. @nickitat Approve? Thanks |
Integration tests (aarch64) [5/6]: #64451 Stress test (tsan): Killed externally (signal 9), some threads hang during shutdown. I believe this problem in tests persist for a long time already, but it is also unrelated to this PR. Upgrade check (debug): Setting A Sync: No merge conflicts, test failures unrelated. --> Forcing |
2680683
As per #60469, CI was upgraded to Clang 18. Clang-tidy checks new in v18 were temporarily disabled.
This PR enables<-- nope, see #64110 (comment)readability-redundant-inline-specifier
.Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
...
Documentation entry for user-facing changes
Modify your CI run
NOTE: If your merge the PR with modified CI you MUST KNOW what you are doing
NOTE: Checked options will be applied if set before CI RunConfig/PrepareRunConfig step
Include tests (required builds will be added automatically):
Exclude tests:
Extra options:
Only specified batches in multi-batch jobs: