Do you perform reviews?

  1. Requirement/Use case review
  2. Architecture review
  3. Security review
  4. Infrastructure review
  5. Design review
  6. Test plan/case review
  7. Compliance/Documentation review
  1. Traditional way —Agree on a mutually convenient time after the code is completed or semi-complete. The developer (who developed/wrote the code) opens up the editor along side one or many more peer developers and explains the code. The peer developers comment on the code. The boundary conditions are picked up and discussed. Traditionally, this type of review is also one of the reasons why peer programming always helps
  2. Remote way — Agree on a mutually convenient time after the code is completed or semi-complete. The developer (who developed/wrote the code) opens up the editor along side one or many more peer developers and explains the code using screen share.
  3. Automated way — The software tools perform the code review in an automated manner. The tool provides a report on possible issues or action that the developer would need to act on. Apply the 80/20 rule. Not all of the tool output would be directly applicable. There would some false positives. Eliminate them. Seek feedback from experts on topics that are not understood (don't struggle alone). Rinse and repeat.
What are the some tools available for use?
GitHub
Collaborator
Review Board
Crucible
Phabricator
SonarQube
  1. Is the code readable?
  2. Are the security checks in place?
  3. Are correct coding constructs used?
  4. Is the code performant?
  5. Identify bugs if any
  6. Is the code testable under positive and negative conditions?

--

--

--

Over two decades of experience in architecture, design, software development. Experience with UML, Java, Web and coaching teams. AWS/Google cloud enthusiast

Love podcasts or audiobooks? Learn on the go with our new app.

Recommended from Medium

You should spend about 20 minutes on this task.

The First AMA with the Sashimi Governance Committee! Ask us anything now!

Automate Your Python Tests With Continuous Integration

What’s happening inside a shell?

Getting Help From Internet While Coding: Part I

Spanish Stock Market Company Profiles Retrieval using investpy

Integrate web-testing into CircleCI jobs with Trakr

Get the Medium app

A button that says 'Download on the App Store', and if clicked it will lead you to the iOS App store
A button that says 'Get it on, Google Play', and if clicked it will lead you to the Google Play store
Sajit Vasudevan

Sajit Vasudevan

Over two decades of experience in architecture, design, software development. Experience with UML, Java, Web and coaching teams. AWS/Google cloud enthusiast

More from Medium

Spend your time in Code Reviews, not in writing Tests

Test Driven Development

Run Automated Tests in Pipeline

The 5 Golden Rules of Code Reviews