PR review "final report box" to show before merging #2579

fregante posted onGitHub

When about to merge PRs:

we could show a final "report" box listing what's about to happen:

full list of commits that will be merged/squashed/rebased, as suggested in https://github.com/sindresorhus/refined-github/issues/2564#issuecomment-557433237
issues/PRs that will automatically be closed (via closes #11 keyword) or if none will be closed
any potential issues to be aware of:
- unrebased !squash/!fixup commits: https://github.com/sindresorhus/refined-github/issues/2385
- reviews that haven't been resolved
- CI that hasn't completed or passed
TODOs added by the PR https://github.com/sindresorhus/refined-github/issues/2634

What else could this include?

full list of commits that will be merged/squashed/rebased

This would be a good start for this feature

posted by fregante about 5 years ago

issues/PRs that will automatically be closed (via closes #11 keyword) or if none will be closed

Actually I'd like to see this more. Sometimes new users forget to mark "closes X" correctly so it'd be good to see "No issues will be closed when merging this PR" as well as "X Y and Z will be closed after merging.

posted by fregante over 3 years ago

Fund this Issue

$0.00

Funded

Only logged in users can fund an issue

Pull requests

Do you want to work on this issue?

PR review "final report box" to show before merging #2579

Company

Community

Support

Connect