Add target to apply clang-tidy fixes

[oliverlee]

No description provided.

[oliverlee]

When I run the example with:

bazel build //example --aspects clang_tidy/clang_tidy.bzl%clang_tidy_aspect --output_groups=report

this silently succeeds. If instead I run

bazel build //... --aspects clang_tidy/clang_tidy.bzl%clang_tidy_aspect --output_groups=report

I get the expected lint warning.

I get tidy fixes applied with

bazel run :apply_fixes

or

bazel run :apply_fixes -- //...

[gou4shi1]

any update?

[MiaoDX]

@oliverlee

Hi, tried to use the fix patch, however, it seems to be confused by sandbox.

I tried to add back no-sandbox inside 99ed470, it succeed in fixing the std::string problem.

dongxu@MiDX01:~/ws/micah/tmp/bazel_clang_tidy$ bazelisk run :apply_fixes //...
INFO: Invocation ID: 2334cead-3d8a-4851-9396-e18a3046da53
INFO: Analyzed target //:apply_fixes (1 packages loaded, 1 target configured).
INFO: Found 1 target...
Target //:apply_fixes up-to-date:
  bazel-bin/clang_tidy.apply_fixes.sh
INFO: Elapsed time: 0.045s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
INFO: Running command line: bazel-bin/clang_tidy.apply_fixes.sh //...
Described file '/home/dongxu/.cache/bazel/_bazel_dongxu/7cea99f25702216cc5bfbca1e10eeac0/sandbox/linux-sandbox/11/execroot/bazel_clang_tidy/example/lib.cpp' doesn't exist. Ignoring...
Described file '/home/dongxu/.cache/bazel/_bazel_dongxu/7cea99f25702216cc5bfbca1e10eeac0/sandbox/linux-sandbox/11/execroot/bazel_clang_tidy/example/lib.hpp' doesn't exist. Ignoring...

[arvid-norlander]

Are there any plans to continue this work?

[oliverlee]

I haven't been working on these things recently so I'm very unlikely to. It may be better to close this issue.

…

On Fri, Mar 7, 2025, 05:22 Arvid Norlander ***@***.***> wrote: Are there any plans to continue this work? — Reply to this email directly, view it on GitHub <#39 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAWT4J4LHPOZZ2IRNKCIKLL2TGMQNAVCNFSM6AAAAABYRGVFIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMBWGQ2DIMRRHA> . You are receiving this because you were mentioned.Message ID: ***@***.***> [image: arvid-norlander]*arvid-norlander* left a comment (erenon/bazel_clang_tidy#39) <#39 (comment)> Are there any plans to continue this work? — Reply to this email directly, view it on GitHub <#39 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAWT4J4LHPOZZ2IRNKCIKLL2TGMQNAVCNFSM6AAAAABYRGVFIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMBWGQ2DIMRRHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>