[GH-ISSUE #2483] Move clang-tidy and cppcheck to a separate CI stage #1218

Open
opened 2026-03-04 01:52:17 +03:00 by kerem · 2 comments
Owner

Originally created by @gaul on GitHub (Jul 1, 2024).
Original GitHub issue: https://github.com/s3fs-fuse/s3fs-fuse/issues/2483

Instead of running clang-tidy and cppcheck across all CI targets, we could put these only in a separate CI target. Running across all targets exposes us to bugs in older versions, e.g., #2477, #2163, #1869, and wastes CI run-time doing duplicate work. Doing this would be similar to the existing valgrind target which only runs on ubuntu-latest.

Originally created by @gaul on GitHub (Jul 1, 2024). Original GitHub issue: https://github.com/s3fs-fuse/s3fs-fuse/issues/2483 Instead of running clang-tidy and cppcheck across all CI targets, we could put these only in a separate CI target. Running across all targets exposes us to bugs in older versions, e.g., #2477, #2163, #1869, and wastes CI run-time doing duplicate work. Doing this would be similar to the existing valgrind target which only runs on ubuntu-latest.
Author
Owner

@gaul commented on GitHub (Jul 1, 2024):

Also shellcheck.

<!-- gh-comment-id:2199155222 --> @gaul commented on GitHub (Jul 1, 2024): Also shellcheck.
Author
Owner

@ggtakec commented on GitHub (Jul 1, 2024):

It seems best to set up separate CIs for each.
It seems best to base them on an OS (distribution) that can support the latest version as quickly as possible.(especially cppcheck)

<!-- gh-comment-id:2200154395 --> @ggtakec commented on GitHub (Jul 1, 2024): It seems best to set up separate CIs for each. It seems best to base them on an OS (distribution) that can support the latest version as quickly as possible.(especially cppcheck)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/s3fs-fuse#1218
No description provided.