Skip to content

fix: resolve issue #295 - bert-e: pr status should be displayed at all time - #296

Closed
matthiasL-scality wants to merge 5 commits into
mainfrom
fix/issue-295
Closed

matthiasL-scality wants to merge 5 commits into
mainfrom
fix/issue-295

fix: address review comments on PR #296

14e92fc
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 1, 2026 in 0s

90.39% (+0.17%) compared to cfe0bc2

View this Pull Request on Codecov

90.39% (+0.17%) compared to cfe0bc2

Details

Codecov Report

❌ Patch coverage is 96.00000% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.39%. Comparing base (cfe0bc2) to head (14e92fc).

Files with missing lines Patch % Lines
bert_e/workflow/pr_utils.py 81.48% 5 Missing ⚠️
bert_e/git_host/base.py 50.00% 1 Missing ⚠️
bert_e/git_host/mock.py 50.00% 1 Missing ⚠️
bert_e/tests/unit/test_status_comment.py 99.36% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #296      +/-   ##
==========================================
+ Coverage   90.21%   90.39%   +0.17%     
==========================================
  Files          82       83       +1     
  Lines       11293    11490     +197     
==========================================
+ Hits        10188    10386     +198     
+ Misses       1105     1104       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.