-
Notifications
You must be signed in to change notification settings - Fork 214
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Merged by Bors] - Fix/6041 bug #6053
Conversation
2. added comments
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #6053 +/- ##
=========================================
- Coverage 81.4% 81.4% -0.1%
=========================================
Files 299 300 +1
Lines 32149 32196 +47
=========================================
+ Hits 26200 26238 +38
- Misses 4238 4245 +7
- Partials 1711 1713 +2 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, I have a few minor comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Minor comment about something that was already there but should be addressed at some point
bors merge |
## Motivation Fix the bug Co-authored-by: ConvallariaMaj <[email protected]>
Build failed (retrying...): |
## Motivation Fix the bug Co-authored-by: ConvallariaMaj <[email protected]>
Pull request successfully merged into develop. Build succeeded: |
Motivation
Fix the bug
Description
Fix of issue #6041
Added timeout to context for searching positioning ATX, wrote couple new tests, extended offloadingPostVerifier.Verify() method to prioritize verificaiton of positioning ATX candidate.
Test Plan