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.

Sign up for GitHub

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

Jump to bottom

improve clang-tidy to support multiple sourcefiles #5592

Merged
merged 2 commits into from
Dec 25, 2024
Merged

improve clang-tidy to support multiple sourcefiles #5592

merged 2 commits into from
Dec 25, 2024
+36 −18

Conversation

waruqi
Copy link
Member

@waruqi waruqi commented Sep 9, 2024

@waruqi waruqi mentioned this pull request Sep 9, 2024
Closed
@waruqi waruqi modified the milestones: v2.9.5, v2.9.6 Sep 9, 2024
@waruqi
Copy link
Member Author

waruqi commented Sep 11, 2024

wait for llvm/llvm-project#103499

@waruqi waruqi removed this from the v2.9.6 milestone Oct 30, 2024
@yh-sb
Copy link
Contributor

yh-sb commented Dec 24, 2024

llvm/llvm-project#103499 was just fixed in scope of llvm/llvm-project#120547.

@waruqi waruqi added this to the v2.9.8 milestone Dec 25, 2024
@waruqi
Copy link
Member Author

waruqi commented Dec 25, 2024

I don't know which version of clang-tidy started to support it, I tested my local 19.1.2 and it seems to work too

@waruqi waruqi merged commit b029e65 into dev Dec 25, 2024
38 checks passed
@waruqi waruqi deleted the tidy branch December 25, 2024 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.9.8
Development

Successfully merging this pull request may close these issues.
2 participants
@waruqi @yh-sb