Fragments 2019

State of engineering, design and product development in the mobile ecosystem

Automation of Pull Request Review

Submitted by Sourabh Gupta (Tech - BLR) (@sourabhflipkart) on Thursday, 22 February 2018

videocam_off

Technical level

Advanced

Status

Submitted

Vote on this proposal

Login to vote

Total votes:  +12

Abstract

Talk will be about how we have added Automation of Pull Request Review using the coding Techniques present in the market and how it will helped us in saving Developer time in Reviewing changes , avoid malicious code to be merged in codebase that causes crashes and results in broken flows.

Outline

Session will be about how we have automated Pull Request Review at Flipkart to make sure new changes that will be merged in the codebase should have -:

-: Proper coding guidelines check using checkstyle.

-: Lint warning Validations.

-: Static code analysis check to figure out NPE , Memory Leaks using FindBugs , PMD.

-: Checks on Apk Size and Method Counts using CI Plugins.

-: Unit Test and E2E test cases validation to make sure app is not crashing post launch and will not break any critical flow.

-: Check on App launch time.

Speaker bio

Mobile Engineer@Flipkart.Leading Flipkart Android Team to create top class Android Apps for Billions of Users in India

Links

Slides

https://medium.com/@sourabhgupta_63169/pull-request-validation-at-flipkart-f84d720cc5ae

Comments

Login with Twitter or Google to leave a comment