Closed wimax-grapl closed 2 years ago
Base: 39.39% // Head: 39.38% // Decreases project coverage by -0.00%
:warning:
Coverage data is based on head (
1d564dd
) compared to base (481bb3a
). Patch has no changes to coverable lines.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
Which issue does this PR correspond to?
https://github.com/grapl-security/issue-tracker/issues/1047
What changes does this PR make to Grapl? Why?
protoc
from source - you have to supply/bundle it. I've added it in three contexts:setup_chromeos
(for people runningcargo
locally on their boxes)rust/build-env.Dockerfile
(formake lint-rust
andmake test-unit-rust
)rust/Dockerfile
(for real building)lastly, I've included the rest of the necessary plumbing to have a single place (
.protoc-version
) to specify version.How were these changes tested?
Passed integration tests on devbox and on CI.