lavanet / lava

Apache License 2.0

404 stars 207 forks source link

feat: PRT - adding debug headers and fixing nil deref on response nil #1651

Closed ranlavanet closed 2 weeks ago

ranlavanet commented 2 weeks ago

Description

Closes: #XXXX

Author Checklist

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues.

I have...

[ ] read the contribution guide
[ ] included the correct type prefix in the PR title, you can find examples of the prefixes below:
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] targeted the main branch
[ ] provided a link to the relevant issue or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] included the necessary unit and integration tests
[ ] updated the relevant documentation or specification, including comments for documenting Go code
[ ] confirmed all CI checks have passed

Reviewers Checklist

All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.

I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic, API design and naming, documentation is accurate, tests and test coverage

github-actions[bot] commented 2 weeks ago

Test Results

2 181 tests ±0 2 181 :white_check_mark: ±0 24m 12s :stopwatch: - 1m 0s 145 suites ±0 0 :zzz: ±0 7 files ±0 0 :x: ±0

Results for commit 3032b2b1. ± Comparison against base commit 395a0541.

:recycle: This comment has been updated with latest results.