Update pull request template#708
Conversation
|
Great job! No new security vulnerabilities introduced in this pull request |
🔍 SDK Breaking Change Detection ResultsSDK Version:
Breaking change detection completed. View SDK workflow |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #708 +/- ##
=======================================
Coverage 79.58% 79.58%
=======================================
Files 293 293
Lines 32767 32767
=======================================
Hits 26078 26078
Misses 6689 6689 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
djsmith85
left a comment
There was a problem hiding this comment.
I actually liked the reminders and the guidelines. I'd regularly look up the emojis when writing reviews. I also think it helps new devs and contributors to see what we are looking out for in a PR.
|
@djsmith85 I found them a bit verbose, but not disagreeing. This was updated in the other repos though so for consistency we should use the same template. Maybe there's a middle ground that won't confuse AI reviewers. |

🎟️ Tracking
📔 Objective
Update PR template to match bitwarden/template#181