Skip to content

Conversation

@srest2021
Copy link
Member

@srest2021 srest2021 commented Jan 19, 2026

fixes CW-283

Depends on https://github.com/getsentry/seer/pull/4539

React with eyes emoji on PR on actions: opened, ready for review, new commit.

@linear
Copy link

linear bot commented Jan 19, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 19, 2026
@srest2021 srest2021 changed the title fix(code review): add eyes reaction to pr feat(code review): add eyes reaction to pr Jan 19, 2026
pr_number = pull_request.get("number")
if pr_number and action in {
PullRequestAction.READY_FOR_REVIEW,
PullRequestAction.OPENED,
Copy link
Member Author

@srest2021 srest2021 Jan 20, 2026

Choose a reason for hiding this comment

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

Is it possible there could be a race condition here--ie, we get two webhooks at the same time, one for OPENED and one for READY_FOR_REVIEW, both see that there's no eye reaction so we get two eye reactions?

If this does happen, the loop over all existing reactions in https://github.com/getsentry/seer/pull/4539 will remove all eye reactions by Sentry bot.

@srest2021 srest2021 marked this pull request as ready for review January 20, 2026 17:09
@srest2021 srest2021 requested a review from a team as a code owner January 20, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants