Are you in the loop? Keep up with the latest by making sure you're subscribed to Community Announcements. Just click Watch and select Articles.

×
Create
cancel
Showing results for 
Search instead for 
Did you mean: 
Sign up Log in
Celebration

Earn badges and make progress

You're on your way to the next level! Join the Kudos program to earn points and save your progress.

Deleted user Avatar
Deleted user

Level 1: Seed

25 / 150 points

Next: Root

Avatar

1 badge earned

Collect

Participate in fun challenges

Challenges come and go, but your rewards stay with you. Do more to earn more!

Challenges
Coins

Gift kudos to your peers

What goes around comes around! Share the love by gifting kudos to your peers.

Recognition
Ribbon

Rise up in the ranks

Keep earning points to reach the top of the leaderboard. It resets every quarter so you always have a chance!

Leaderboard

Bitbucket pull request merge

Yogesh Mude
Rising Star
Rising Star
Rising Stars are recognized for providing high-quality answers to other users. Rising Stars receive a certificate of achievement and are on the path to becoming Community Leaders.
Jul 14, 2020

Hi Team,

The situation is...whenever the developer finished his/her development work, developer creates a pull request in bitbucket and adds the reviewer to review it. The reviewer reviews it and provides feedback and approve the pull request.

After approving the pull request, the reviewer merges the pull request and bitbucket uses the reviewer name as the author of the commit.

So my question here is, commit author should be a developer as he/she did the code changes but not a reviewer or the person did the pull request merge.

Thanks in advance.

1 comment

Ulrich Kuhnhardt _IzymesCo_
Marketplace Partner
Marketplace Partners provide apps and integrations available on the Atlassian Marketplace that extend the power of Atlassian products.
Jul 14, 2020

Hi Yogesh

I understand that you need a developer persona to be the author of commits including merge commits. 

My suggestion is to change your workflow and to have the actual author of the PR perform the merge once approvals are given. 

 

You can add simple merge checks I.e number of required approvals to ensure PRs are not merged before approvals are given. If you have more complex workflow requirements you can use Bitbucket apps that help you define your process and enforce it. Let me know if you have any questions 

Yogesh Mude
Rising Star
Rising Star
Rising Stars are recognized for providing high-quality answers to other users. Rising Stars receive a certificate of achievement and are on the path to becoming Community Leaders.
Jul 15, 2020

HI @Ulrich Kuhnhardt _IzymesCo_ 

Thanks for the response.

As per the below articles ...PR merge is the flexibility that can allow different team members who have the write permission for the repository can merge. 

https://confluence.atlassian.com/bitbucketserver/merge-a-pull-request-808488562.html

Also, I have the merge checks, which we are checking all reviewers approval required and the PR merge is done once the reviewer is approved. 

Sometimes the reviewer itself merges the PR, in this case, the commit author should be added as a person who opens the PR (ie. developer) but not the reviewer. 

How I can do this.

Comment

Log in or Sign up to comment
TAGS
AUG Leaders

Atlassian Community Events