Skip to content

Conversation

@colbylwilliams
Copy link
Contributor

@colbylwilliams colbylwilliams commented Mar 26, 2025

Following up on changes in #3532

This PR adds List and Delete functionality to reactions associated with a release

@codecov
Copy link

codecov bot commented Mar 26, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.31%. Comparing base (51d32d9) to head (7193318).
Report is 1 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@ ## master #3533 +/- ## ========================================== + Coverage 91.30% 91.31% +0.01%  ========================================== Files 184 184 Lines 16143 16169 +26 ========================================== + Hits 14739 14765 +26  Misses 1230 1230 Partials 174 174 

☔ View full report in Codecov by Sentry.
📢 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.
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor tweak, please, @colbylwilliams, then we should be ready for a second LGTM+Approval from any other contributor to this repo before merging.

@stevehipwell - might you have time for a code review? Thank you!

@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Mar 26, 2025
Copy link
Contributor

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gmlewis gmlewis removed the NeedsReview PR is awaiting a review before merging. label Mar 27, 2025
@gmlewis
Copy link
Collaborator

gmlewis commented Mar 27, 2025

Thank you, @stevehipwell!
Merging.

@gmlewis gmlewis merged commit 4f05dfb into google:master Mar 27, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants