eclipse-score / eclipse-score/communication

Improvement: Check that PRs to be merged have no merge commits

Open
#800 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
62
Forks
97
Avg merge
1d 1h
Merged PRs (30d)
72

Description

### What

When creating merge commits, conflicts might arise. If that is the case, and conflicts need to be resolved, it might be hard to see in a pull request what was change. Because of that, to reduce the chance that unintentional changes are merged, we should forbid merge commits in pull requests.

### How

_No response_

### Estimates for realization

_No response_

### Category

- [ ] Affects Detailed Design

### Requirements / Architecture

- [x] Requirements / Architecture are not affected by this change?

Contributor guide

Open the contributing guide

Research direction

Start by inspecting the repository’s GitHub configuration and workflow files for pull-request merge rules. Reproduce a pull request containing a merge commit and identify the existing validation or protection entry point. Done means such pull requests are reliably rejected or blocked, with the behavior verified through the relevant GitHub check or configuration.

Written by the indexing model from the issue text.

Assessment

Tech stack
git, github
Domain
ci-cd
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.