eclipse-score / eclipse-score/communication
Improvement: Check that PRs to be merged have no merge commits
- 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
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