bazel_clang_tidy icon indicating copy to clipboard operation
bazel_clang_tidy copied to clipboard

Published 20 hours ago • verified publisher icon erenon

Some fixes for macos, objective C, etc.

Open binji opened this issue 3 years ago • 2 comments

Hey all!

I made some local changes to get this to work with our macOS toolchain. The PR can't be uploaded like this, but I'm happy to work with you to get it into shape so it can. Just let me know!

binji avatar Sep 14 '22 23:09 binji

hi, thanks for the effort. unfortunately, in the foreseeable future, I'll have no capacity to work in this.

erenon avatar Sep 15 '22 05:09 erenon

No problem! Just to be clear, are you not looking for PRs for this repo, or are you only landing PRs that are closer to being landed? I just ask since I can clean it up independently too, but I thought maybe you'd have some opinions on the design.

binji avatar Sep 15 '22 22:09 binji

I'm happy to accept PRs, with the condition of previously working features should keep working. I'm also here to review - tough don't expect too much valuable insight - I have little macOS experience : )

erenon avatar Sep 19 '22 07:09 erenon