Hi,
I am wondering about some Pull-Request features and settings that I would find very useful. Some of them are inspired by existing features from gitlab merge-requests.
- Notion of comment threads:
On Gitlab, when a reviewer makes a new comment it opens a new comment thread. This thread needs to be actively closed. Otherwise the PRs will not be mergeable. So for example a team could follow a protocoll, where is the reviewer closes the thread when he is happy with the discussion or the code change. - Code suggestions:
With our current version of bitbucket we cannot do code changes. I heard that the feature exists on bitbucket too. However unsure on how to enable it.
In Gitlab it is possible to give some code changes that can be applied to the branch with one click. And then event batch them into a single 'feedback' commit. This is useful for fixing minor changes e.G. typos. - Setting to automatically unapproving a PR once a has been applied ontop of an already approved PR.
- Setting to no mark commet as outdated by default if diff is applied to a line.
For example, a reviewer adds a comment to a line. The person who opened the PR, makes a new commit with breaking changes. The comment is automatically marked as 'outdated' and appears irrelvant. Although the code chage is not valid at all. Its cumbersome for the reviewer to go through outdated comments to check which ones are still valid and which ones are not.
It would be useful to know if those features and settings already exists somewhere ?
What the community thinks of these ?
Or wont be expected to exists in the future for a given reason ?