Closed tcodehuber closed 1 month ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 27.47%. Comparing base (
890640b
) to head (987af27
). Report is 8 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@tcodehuber I left some small suggestions. PTAL.
Done.
Why are the changes needed?
Close #2897.
Brief change log
-
How was this patch tested?
[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
[ ] Add screenshots for manual tests if appropriate
[ ] Run test locally before making a pull request
Documentation