Use cquery's --output_file if supported#115
Conversation
fmeum
force-pushed
the
main
branch
2 times, most recently
from
8391e50 to
fc8f06b
Compare
|
This PR requires an update to the branch protection rules. |
|
Looks like there's some weirdness going on with that non-matrix presubmit job (which doesn't show up in the Actions tab). I'll merge as is, assuming that it's a Github bug, and that it won't happen in a new PR. |
|
@fmeum before I merge, do you mind squashing commits 1 and 3? |
Bazel command output is included in the BES, which can cause excessive BES traffic for large cquery results. Instead, use the `--output_file` option introduced in 8.2.0.
Done!
That's expected: branch protection matches actions by name, so it's still looking for the old one with its non-matrix name. You have to update the branch protection rule to look for the two new jobs. |
|
Thanks @fmeum, now I understand this comment :-) |
3 participants
Bazel command output is included in the BES, which can cause excessive BES traffic for large cquery results. Instead, use the
--output_fileoption introduced in 8.2.0.Also test in a matrix with all supported Bazel versions, which requires changes to the protos under
third_party.