Skip to Content
Technical Articles
Author's profile photo Christoph Pohl

ABAP Code Review Guideline is out

A new guide on ABAP Code Reviews was added to the SAP Code Style Guides repository, which we want to briefly highlight in this blog post.

What’s it all about?

Change-based peer code reviews increase code quality by finding defects earlier and preventing them from polluting the main code line. They also enable continuous integration by registering automated checks as reviewers. These modern capabilities have been on the wish list of ABAP developers for some time.

ABAP%20Code%20Reviews

Lars Hvam and I have surveyed current possibilities of applying git-based review platforms to ABAP code and give recommendations for authors and reviewers. We came up with a guideline and contributed it to the SAP Code Style Guides, an open source, grassroots project driven by the SAP community. You may know this project, for instance, from Klaus Haeuptle‘s recent blog post on Clean ABAP.

Where can I get it?

You can directly read it online.

What’s in it for me?

The guide discusses how to apply well-known code review flows as known from other programming languages also to ABAP.

Code%20Review%20Flow

We start with previous work like the following:

Then we suggest a number of code review best practices comparable to Google’s Engineering Practices documentation, before we survey existing tools like Git, GitHub, CTS, abapGit and gCTS, ATC and abapLint, and their extension points.

We finally sketch different collaboration scenarios, in which git-based code reviews can be set up for ABAP development projects and give examples for those, using abapGit and gCTS respectively.

How can I give feedback?

As always with open-source projects, contributions are welcome. Please do not hesitate to raise any issues. We love to hear from you!

 

Assigned Tags

      10 Comments
      You must be Logged on to comment or reply to a post.
      Author's profile photo Mohinder Singh Chauhan
      Mohinder Singh Chauhan

      Thanks for Sharing.

       

      Regards

      Mohinder

      Author's profile photo Jacek Wozniczak
      Jacek Wozniczak

      👏 and very nice that you put it into official style guides repo

      Author's profile photo Klaus Haeuptle
      Klaus Haeuptle

      Thanks for the contribution to Clean ABAP!

      Author's profile photo SRINIVASULU BESTA
      SRINIVASULU BESTA

      Thanks for sharing.. useful info..

       

      Regards,

      Srinivas

      Author's profile photo Johannes Gerbershagen
      Johannes Gerbershagen

      Pull request feature for https://github.com/larshp/abapGitServer is coming soon. So reviews for private projects can be done directly on AS-ABAP.

      Author's profile photo Lars Hvam
      Lars Hvam

      Cool, this means the code does not have to leave the actual ABAP system, while still using some git features. abapGitServer is not optimized for large codebases(yet), but its open source and everyone can try it out 🙂

      Author's profile photo Javier Andres Caceres Moreno
      Javier Andres Caceres Moreno

      Good

      Author's profile photo Jelena Perfiljeva
      Jelena Perfiljeva

      Thanks for sharing this, Christoph! This sparked many conversations in the community and we even dedicated an episode of our Mindset Nebula podcast to the code reviews. 🙂 It was a pleasure to host Lars Hvam as our special guest for this conversation. Can you imagine Lars actually does code review for some very lucky developers? 🙂

      Author's profile photo Aman Singh
      Aman Singh

      Thanks for the contribution 🙂

      Author's profile photo Christoph Pohl
      Christoph Pohl
      Blog Post Author

      FYI: The guide has now been converted to previewable markdown : https://github.com/SAP/styleguides/blob/main/abap-code-review/ABAPCodeReview.md