Skip to content
Snippets Groups Projects

mrhlpr: use Part-of trailer instead of adding MR ID to subject

Merged Caleb Connolly requested to merge caleb/use-part-of into master
All threads resolved!

We are planning to migrate to marge-bot for handling merging commits, and with it adopt a new way of linking commits to their source MRs.

Using a trailer with Part-of: !MR-ID results in the commits being linked to the MR, and the GitLab web interface inserting a hyperlink in its place making it easier to jump to the source MR.

Let's adopt this for MRs we still merge manually now.

Use the full URL to the MR rather than just the ID, this way if we ever split or merge repos it will still be easily possible to trace commits back to their original MRs.

Signed-off-by: Caleb Connolly caleb@postmarketos.org

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • achill (fossdd) approved this merge request

    approved this merge request

  • Robert Eckelmann approved this merge request

    approved this merge request

  • Small thing, I'll just amend before merge. Thanks!

  • added 1 commit

    • 4a7d460f - mrhlpr: use Part-of trailer instead of adding MR ID to subject

    Compare with previous version

  • Pablo Correa Gomez resolved all threads

    resolved all threads

  • Caleb Connolly mentioned in commit 55196b21

    mentioned in commit 55196b21

  • added 1 commit

    • 55196b21 - mrhlpr: use Part-of trailer instead of adding MR ID to subject

    Compare with previous version

  • Pablo Correa Gomez approved this merge request

    approved this merge request

  • Caleb Connolly mentioned in commit 7c5ee23d

    mentioned in commit 7c5ee23d

  • added 1 commit

    • 7c5ee23d - mrhlpr: use Part-of trailer instead of adding MR ID to subject

    Compare with previous version

  • Pablo Correa Gomez enabled an automatic merge when all merge checks for 7c5ee23d pass

    enabled an automatic merge when all merge checks for 7c5ee23d pass

  • Please register or sign in to reply
    Loading