nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

how to enable pull request and code review?

Eliah Ninyo

Hello,

currently my repository doesn't enforce pull requests.

where can i set mandatory pull request? to prevent pushes with no permission.

i want also to add the process a code review. is there a plug-in for that?

in addition, as you realized, there were some pushes over the time and now i want to code review them. since it doesn't include pull request (since it is already in), what should i use? any suggestions?

thanks in advance,

Accepted answers

Tyler T

Hi @Eliah Ninyo ,

You can use branch permissions to enforce pull requests and add other restrictions to the master branch (for example). You can prevent rewriting of master and force pushes, etc.

Here is the documentation: https://confluence.atlassian.com/bitbucket/branch-permissions-385912271.html

All comments

Tyler T

Hi @Eliah Ninyo ,

You can use branch permissions to enforce pull requests and add other restrictions to the master branch (for example). You can prevent rewriting of master and force pushes, etc.

Here is the documentation: https://confluence.atlassian.com/bitbucket/branch-permissions-385912271.html

Eliah Ninyo

Thanks Tyler,

i will read the link you sent.

can you please answer also the other questions regarding code review suggestions?

Tyler T

This question?

there were some pushes over the time and now i want to code review them. since it doesn't include pull request (since it is already in), what should i use? any suggestions?

You might need to review commits individually based on the author. I don't have a great suggestion for that. Installing a plugin like GitLens into your code editor might be helpful since you will be able to see who viewed commits as you encounter them in your code.

Eliah Ninyo

i put in the "merge via pull request" all the group "Everyone".

but this doesn't enfore pull request.

people can still do push without pull requests.

any other suggestions?

Tyler T

Is 'write access' empty?

Eliah Ninyo

oh, you are right.

i should remove the write access too :-)

thanks

Welcome!

It looks like you're new here. Sign in or register to get started.

Quick Links

Find more posts tagged with

cloud

bitbucket

bitbucket-cloud