Skip to main content
Solved

Code review per project


Forum|alt.badge.img+4

Hi,

I read the ‘Code review’ blog but I think this will be mandatory for each project. But is it possible to set ‘Code review required’ for specific projects in Software Factory?

Best answer by Mark Jongeling

Kasper Reijnders wrote:

According to the documentation this should be per model/project

https://docs.thinkwisesoftware.com/docs/sf/code_review

I see that this is misleading, I'll make sure it will be updated. Turning off the setting turns off Code review for the complete Software Factory. It is currently not possible to have some models/branches with and some without Code review.

Having this setting on Model level can be an Idea 😄

View original
Did this topic help you find an answer to your question?
This topic has been closed for comments

4 replies

Kasper Reijnders
Forum|alt.badge.img+5

According to the documentation this should be per model/project

https://docs.thinkwisesoftware.com/docs/sf/code_review


Mark Jongeling
Administrator
Forum|alt.badge.img+23
  • Administrator
  • 3936 replies
  • Answer
  • February 6, 2023
Kasper Reijnders wrote:

According to the documentation this should be per model/project

https://docs.thinkwisesoftware.com/docs/sf/code_review

I see that this is misleading, I'll make sure it will be updated. Turning off the setting turns off Code review for the complete Software Factory. It is currently not possible to have some models/branches with and some without Code review.

Having this setting on Model level can be an Idea 😄


Kasper Reijnders
Forum|alt.badge.img+5
Mark Jongeling wrote:
Kasper Reijnders wrote:

According to the documentation this should be per model/project

https://docs.thinkwisesoftware.com/docs/sf/code_review

I see that this is misleading, I'll make sure it will be updated. Turning off the setting turns off Code review for the complete Software Factory. It is currently not possible to have some models/branches with and some without Code review.

Having this setting on Model level can be an Idea 😄

Whoops, I actually first typed "as far as I know this is not possible now.” But then looked at the docs indeed. Got mislead. 


Forum|alt.badge.img+4

Allright. Thanks for clearing that up!


Cookie policy

We use cookies to enhance and personalize your experience. If you accept you agree to our full cookie policy. Learn more about our cookies.

 
Cookie settings