Closed cmaglie closed 3 months ago
Attention: Patch coverage is 33.33333%
with 2 lines
in your changes are missing coverage. Please review.
Project coverage is 69.56%. Comparing base (
6faca0f
) to head (fecb334
). Report is 1 commits behind head on master.:exclamation: Current head fecb334 differs from pull request most recent head 16e1ba6. Consider uploading reports for the commit 16e1ba6 to get more accurate results
Files | Patch % | Lines |
---|---|---|
commands/compile/compile.go | 0.00% | 1 Missing and 1 partial :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Please check if the PR fulfills these requirements
See how to contribute
UPGRADING.md
has been updated with a migration guide (for breaking changes)configuration.schema.json
updated if new parameters are added.What kind of change does this PR introduce?
The optional gRPC field
export_binaries
now leverage theoptional
clause in latest version of protobuf 3 protocol.What is the current behavior?
No change
What is the new behavior?
No change
Does this PR introduce a breaking change, and is titled accordingly?
Yes
Other information
related to #2565