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

fix compatibility with clang-format 18 #6690

Merged
merged 2 commits into from
Mar 15, 2024

Conversation

Coeur
Copy link
Collaborator

@Coeur Coeur commented Mar 14, 2024

✗ clang-format --version
clang-format version 18.1.1

@Coeur
Copy link
Collaborator Author

Coeur commented Mar 14, 2024

Hum, not compatible with clang-17. Let me try to add a comment to the line.
[edit]
Now it works.

@Coeur Coeur changed the title fix clang-format fix compatibility with clang-format 18 Mar 14, 2024
@transmission transmission deleted a comment from GaryElshaw Mar 15, 2024
@ckerr
Copy link
Member

ckerr commented Mar 15, 2024

Whatever the issue with sanitizer-tests is, it's affecting multiple PRs and is unrelated to the changes in 6690

@ckerr ckerr merged commit edc59ba into transmission:main Mar 15, 2024
21 of 22 checks passed
@Coeur Coeur deleted the coeur/clang-format branch March 15, 2024 06:48
Coeur added a commit to Coeur/swiftmission that referenced this pull request Mar 15, 2024
Schlossgeist pushed a commit to Schlossgeist/transmission that referenced this pull request Mar 23, 2024
* fix clang-format

* compatibility with clang-format 17-18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants