Code review helps to spread knowledge and best practices around code across the team. It helps the team to grow as a unit. Some teams grow horizontally like front end team, database team and server management team, whereas other grow vertically on specific product features. Code reviews can help all kind of teams to grow and learn across the code base.
Why do we need code review
- A specific feature expert in the product can end up with all of the responsibility of the feature and if they leave or take a vacation then a major problem is created to maintain and manage that feature.
- Onboarding of new team members
- Discovery of bugs
- Better performing code
How to achieve it
- Code review should be a separate step in the sprint workflow, so that its visible to everyone.
- If required a waiting state can be added before code review, like waiting for code review, so that a bottleneck can be highlighted.
- QA team should be involved in the code review. ( Not sure if our QA team is able to do that with their Java experience )
- Code reviews should be done in a non-disruptive way, asynchronously when its a natural stopping point for the developer and reviewer.
- Code review feedback should be passed through an automated tool like version control system (Copy to JIRA for reference), so that it is easily managed along with the code. ( I have installed the subversion plugin to JIRA, so that we can associate our SVN comments into JIRA )
On Windoze one can install assembles into Global assembly cache to make assembly accessible from everywhere on the machine, there are four main ways to install assembly into global assembly cache , but the one that I found to be the easiest and available by default on windows is the Assembly Cache Viewer (Shfusion.dll) , using this method one can use windows explorer to drag the assembly file into the global cache folder at windows\assembly however this method requires that the assembly is strongly typed, in order to do that one has to sign the assembly, In Visual studio this can be done by click on the class library project properties and select Signing tab and then selecting Sign the assembly.
Note: On Windows Vista I was getting Access denied message on trying to use Assembly Cache Viewer although I was performing the copy using Administrative rights, and ended up using .NET Framework Configuration Tool (Mscorcfg.msc) which worked like a charm for me. So if you are getting the same issue better try using the alternate ways of installing the assembly .