coala / coala/meta

How about letting developers ack PRs?

Open
#112 11 comments 6 reactions 0 assignees View on GitHub
enhancement
Dominant language
No language data
Stars
6
Forks
5
PR merge metrics
No merged PRs in 30d

Description

These PRs can go in a separate queue for maintainers and will definitely not contain trivial errors but may contain design changes. Gitmate can have one more state if PRs are acked by a developer, somewhat like pseudo passed but still not approved for merge.

The more developers ack a PR, the PR can get to one side of the queue, that means it has more probability of being error free. So if someone is not quite in the mood to review, he can start from this end and keep merging.

While those PRs which are acked by lesser developers or (some requested changes and some acked) should go to the other end, as a maintainer will have to look closely and then see why is the PR being controversial.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the discussion on issue #112, including the comments about developer acknowledgements, queue ordering, and maintainer review. Done would require an agreed design for the acknowledgement state and queue behavior, followed by the corresponding implementation or documented decision.

Written by the indexing model from the issue text.

Assessment

Domain
developer-experience
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.