All geek questions in one place

Prevent commit before peering in svn

Is there any svn hook for peer review? I would like to make sure that any developer will not be able to execute the code until it has been verified by the peer. How can this be done using svn?

+6
source share
2 answers

I feel a misunderstanding here: if you are trying to add a code review as a preliminary verification step, you consider the source repository as authority for the perfect, perfect code, but do not allow its imperfect code. This is detrimental to the overall usefulness of the original repository. The code should be in storage as early as possible. Code in the repository is tracked with changes, easily accessible, easier to view, and simplifies backups.

So, follow the easy route: save the code in the branches (maybe an additional unreviewed folder next to branches and tags ) and look at the branch before it is merged into its target branch. The technical side becomes simpler, and you do not need to intercept hooks.

+4
source

What you need is a code checking tool that works with svn. See some suggestions here: What is the best way to review code using Subversion (SVN)? [Closed]

Also see this very similar Quora question that has good answers: What are the best code checking tools?

+1
source

Source: https://habr.com/ru/post/906270/

More articles:


All Articles