Unify our linting strategy with aspect's rules_lint for linting #125
Comments
michael-christen
changed the title
Merged
michael-christen
added a commit
that referenced
this issue
Nov 10, 2024
Start to setup clang-format using rules_lint: https://github.com/aspect-build/rules_lint We should have a followup to consolidate our linting / formatting tools. this does get us setup for C++ (which will be enabled in a followup). See #125 Also, added a `--config local` for bazel to avoid remote caching, etc. to avoid network latency when my home network is slow.
michael-christen
added a commit
that referenced
this issue
Nov 12, 2024
This is the culmination of several PRs, allowing us to bind this service to this app. See #155 (comment) for my log of notes along the way. Here are the related PRs: - #153 - #152 - #151 - #150 - #143 - #142 - #141 - #140 - #138 And the created/related issues: - #154 - #149 - #148 - #147 - #145 - #144 - #139 - #125
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://blog.aspect.build/lint
https://github.com/aspect-build/rules_lint
(2024-11-10)
We're starting to use this in #138. This issue should switch to tracking the work necessary to "unify" our linting approaches.
The text was updated successfully, but these errors were encountered: