Sync bot crushing partial reviews
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 30/100
Research direction
No files, tests, or entry points are named. Start by locating the sync bot's branch and pull-request handling, then reproduce a sync with a partial review; done means synchronization no longer force-pushes over reviewed work and preserves the review context.
Written by the indexing model from the issue text.
Description
Hi @iliakan
I discovered this in the Spanish repo,
but happens in the other repos too.
...
Previously, the bot used different branches for sync, closing the old PR and generating a new one over a new branch. Partial reviews were preserved in the closed PR.
Now, the bot closes the old PR but always uses the same branch, doing a force push on it that destroys any partial reviews.
- Dominant language
- HTML
- Stars
- 25.5k
- Forks
- 4k
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from javascript-tutorial/en.javascript.info
-
Difficulty 1/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
-
Difficulty 1/5 Under an hour Newbie friendliness 92/100
-
Keyboard events: keydown filtering example can't block IME-composed input (Korean/Japanese/Chinese) Open
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
javascript-tutorial/en.javascript.info#3978 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
All issues in javascript-tutorial/en.javascript.info
Similar issues
-
Add dependabot Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 1/5 Under an hour Newbie friendliness 75/100
-
oblt-aw/detector/security
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Type: AVM :a: :v: :m: Type: Bug :bug:
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Azure/bicep-registry-modules#7386 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100