Skip to main content
Answer

Code review per project

  • February 6, 2023
  • 4 replies
  • 56 views

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

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 😄

This topic has been closed for replies.

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
  • Answer
  • February 6, 2023

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

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!