Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bump rules_cc to 2f8c04c #130

Merged
merged 1 commit into from
Aug 24, 2022

Conversation

shaldengeki
Copy link
Contributor

This is my first contribution to rules_proto -- greatly appreciate your patience! I've already signed the CLA on my own behalf.

Fixes #95.

This bumps rules_cc to (as of writing) HEAD, authored ~21 days ago:
bazelbuild/rules_cc@2f8c04c

To verify, I:

ouguoc:~/bazel_clang_tidy$ bazel build //example:app     --aspects clang_tidy/clang_tidy.bzl%clang_tidy_aspect     --output_groups=report
INFO: Analyzed target //example:app (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
INFO: From Run clang-tidy on example/app.cpp:
3005 warnings generated.
Suppressed 3005 warnings (3005 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
Aspect //clang_tidy:clang_tidy.bzl%clang_tidy_aspect of //example:app up-to-date:
  bazel-bin/example/example/app.cpp.app.clang-tidy.yaml
INFO: Elapsed time: 2.845s, Critical Path: 2.65s
INFO: 2 processes: 1 internal, 1 local.
INFO: Build completed successfully, 2 total actions

@comius comius merged commit 6b5ba87 into bazelbuild:master Aug 24, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update rules_cc dependency to more recent commit
2 participants