Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G gitlabhq1
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 12
    • Merge requests 12
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • gpt
  • large_projects
  • gitlabhq1
  • Merge requests
  • !2384

Closed
Created Dec 25, 2012 by Administrator@rootOwner
  • Report abuse
Report abuse

Add alternative merge type and merge notes to merge requests

  • Overview 10
  • Commits 2
  • Changes 3

Created by: calmh

This adds two minor, optional features to merge requests that were necessary for our workflow and might be useful for others.

"merge_ff_only": Instead of the default "pull --no-ff" that creates a merge commit, do "merge --ff-only" on merge requests. This means the resulting history on the destination branch will be linear, and also that non-ff merge requests will be marked as unmergeable until rebased.

"add_merge_notes": With the above strategy we won't get the merge commit indicating who did the merge. Instead, this adds notes to refs/notes/merge to each merged commit with the text "Merge-By: Full Name [email protected]" which can be pulled and examined after the fact.

I'll be keeping this as a local patch either way, so if there's no interest from the community for these features, feel free to close the pull request.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: github/fork/calmh/alt-merge