Repository navigation
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7551 +/- ##
=======================================
Coverage 86.82% 86.82%
=======================================
Files 896 896
Lines 65792 65792
Branches 9407 9407
=======================================
Hits 57121 57121
Misses 5187 5187
Partials 3484 3484
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
🤖 Bitwarden Claude Code ReviewOverall Assessment: APPROVE This PR adds a |
vvolkgang
left a comment
There was a problem hiding this comment.
Reviewed in the iOS repo: bitwarden/ios#3177 (review)
🎟️ Tracking
PM-46111
📔 Objective
Add a "🚩 Flags removed" category to the generated release notes for PRs labeled
t:flagged-code-removal. Without it, those PRs fall through to "🎨 Other".The category sits below Maintenance and above Dependency Updates. Cleanup PRs carry only this
t:label, so no earlier category claims them.This will allow feature flag removals (which unlock features for self-hosted customers) to be categorized independently in release notes. Note that this is indicating the removal of flagged code and not the feature flag itself, which must remain in place for backward compatibility.