DEV Community

Joyce Wei
Joyce Wei

Posted on

SPO600 Lab1 - Code Review

The open source projects required a license which specifies the authorization that everyone in the community has in order to contribute and make change to the code.

In this lab, I will be look into two open source projects each with different licenses and observe the code review process on how the contributors can contribute to the project.

DuckDuckGo iOS Application

DuckDuckGo is an internet search engine that licensed under the Apache 2.0 license.

To contribute to this project, contributors can either report an issue or require to create a pull request with a fix in their GitHub page with detailed description stated. Once one of the reviewers has reviewed the code and approved the changes, the code will be merge into the main branch.

There are two participants in this pull request, one is the contributor and one is the reviewer. The whole process take up 3 days to add a JS prompts support to the application. The contributor creates the pull request first, then the reviewer will review the code and provide some feedbacks. Once the code is fixed and approved, the code will be merged into the main branch.

Mikro-Orm

Mikro-Orm is TypeScript ORM for Node.js application that licensed under the MIT license.

To contribute to this project, contributors can either report an issue or require to create a pull request with a fix in their GitHub page with description stated. Once the reviewer has reviewed the code and approved the changes, the code will be merge into the main branch.

There are two participants in this pull request, one is the contributor and one is the reviewer. The whole process only takes up less than an hour to fix a missing type issue. The contributor creates the pull request first, then the reviewer will review the code and request changes to the fix code. Once the code is fixed and approved, a code coverage comment bot will come and show the code coverage result in the comment then the fix code will be merged into the main branch.

Top comments (0)