matrix-org / matrix-org/mscbot-python

Raising multiple concerns in a single review races

Open
#31 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
7
Forks
2
PR merge metrics
No merged PRs in 30d

Description

1. "Add review comment" for your concern
2. Add a second concern
3. Submit your review
4. mscbot updates comment twice, overwriting values.

![image](https://user-images.githubusercontent.com/1190097/212772584-5377851c-9e56-476a-8670-66b09a0c7684.png)

Happened from https://github.com/matrix-org/matrix-spec-proposals/pull/2746#pullrequestreview-1250691205

Contributor guide

Open the contributing guide

Research direction

Reproduce the issue using the four-step sequence in the report, then trace mscbot-python’s handling of GitHub pull-request reviews and the two updates triggered by multiple concerns. Done means each concern’s values remains intact after the review is submitted; the issue names no source file or test to start from.

Written by the indexing model from the issue text.

Assessment

Tech stack
github, python
Domain
devtools
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
38/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.