alexellis / alexellis/derek

Proposal: Label and message for PRs with Merge Commits

未关闭
#151 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
Go
星标
820
派生
68
PR 合并指标
30 天内没有已合并 PR

描述

If there's a merge commit pushed to a PR can we add a label and post a message to the PR from Derek saying that Merge Commits are not allowed, and provide info on how to remove it, and rebase.

Can we detect easily merge commits? Is it the format of the commit message or are they different?

## Expected Behaviour

A PR with a merge commit pushed to it should get a label from derek and a message posted about why and how to fix it.
2 scenarios
1) A PR is raised and there is already a merge commit
2) A Merge Commit is pushed to the PR

Should this be optional? Configured in the yaml? What about for repos that dont have the config set? Do we enable or disable by default?

## Current Behaviour

No feature
## Possible Solution

Add a check into Derek that fulfils the 2x scenarios if enabled.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。