Skip to content

Add clang-tidy in clang images#95

Merged
mathbunnyru merged 2 commits intoXRPLF:mainfrom
kuznetsss:Add_clang_tidy
Feb 12, 2026
Merged

Add clang-tidy in clang images#95
mathbunnyru merged 2 commits intoXRPLF:mainfrom
kuznetsss:Add_clang_tidy

Conversation

@kuznetsss
Copy link
Contributor

Also add clang-21 images.

Copy link
Collaborator

@mathbunnyru mathbunnyru left a comment

Choose a reason for hiding this comment

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

You either need to provide separate LLVM_VERSION, or remove it

@mathbunnyru mathbunnyru merged commit 3c1da1d into XRPLF:main Feb 12, 2026
20 checks passed
@bthomee
Copy link
Contributor

bthomee commented Feb 12, 2026

I propose to revert this change, since we don't need clang-tidy in all Ubuntu images. Instead, I'd like you to create a separate tools-rippled-clang-tidy image for this. We already have similar tools-rippled-clang-format, for instance.

@kuznetsss
Copy link
Contributor Author

kuznetsss commented Feb 12, 2026

@bthomee, but we need all the tools from a regular image to be able to generate compile_commangs.json. Creating a separate image will require to make another duplication of docker file. But having clang-tidy in each image doesn't do any harm except small size increase.

@kuznetsss kuznetsss deleted the Add_clang_tidy branch February 13, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants