Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mention related MRs in MR descriptions #31

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

dancysoft
Copy link
Collaborator

@dancysoft dancysoft commented Aug 29, 2023

When more than one commit is pushed at once, update the description of
all associated MRs to include a list of related MRs. This makes it
easy to see what MRs are part of the same unit of work.

Sample description:

Related MRs:

  • !23 (This MR)
  • !25
  • !24
  • production

@dancysoft dancysoft force-pushed the review/dancy/mention-relation-chain-in-mr-desc branch 2 times, most recently from 2bc9cae to 808f493 Compare August 30, 2023 22:54
@dancysoft dancysoft changed the title WIP: mention related MRs in MR descriptions Mention related MRs in MR descriptions Aug 30, 2023
@dancysoft dancysoft force-pushed the review/dancy/mention-relation-chain-in-mr-desc branch 2 times, most recently from 5c2926d to 5b33b72 Compare August 31, 2023 14:53
@dancysoft
Copy link
Collaborator Author

Copy link
Owner

@yaoyuannnn yaoyuannnn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for implementing this much-needed feature! Much appreciated!

gerritlab/main.py Outdated Show resolved Hide resolved
gerritlab/main.py Outdated Show resolved Hide resolved
@dancysoft dancysoft force-pushed the review/dancy/mention-relation-chain-in-mr-desc branch from 5b33b72 to aa7bb35 Compare August 31, 2023 17:21
When more than one commit is pushed at once, update the description of
all associated MRs to include a list of related MRs.  This makes it
easy to see what MRs are part of the same unit of work.

Sample description:

Related MRs:

* !23 (This MR)
* !25
* !24
* _production_

Change-Id: I7e1da3a6c2c24477498ccc73e38409e7fd2eaf05
@dancysoft dancysoft force-pushed the review/dancy/mention-relation-chain-in-mr-desc branch from aa7bb35 to 7aeedcf Compare August 31, 2023 17:22
Copy link
Collaborator

@thcipriani thcipriani left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it! Works great!

@thcipriani thcipriani merged commit 93bdda8 into main Aug 31, 2023
@dancysoft dancysoft deleted the review/dancy/mention-relation-chain-in-mr-desc branch August 31, 2023 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants