Closed cisse21 closed 1 day ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 91.66667%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 74.37%. Comparing base (
d1a1186
) to head (79a6fba
). Report is 2 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
processor/transformer/transformer.go | 90.90% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description
Add missing
sentStat
intransformer.go
Right now if the response from transformer has mismatch in the contract we expect we end up discarding those jobs to proc errors but ideally there should be no breach in contract and if there is a breach in contract we should stop processing and fix the same. Since the current code structure does not allow propagating errors, adding a panic to stop processing
Linear Ticket
Fixes PIPE-1248
Security