iP:
A-Gradle
, A-CheckStyle
, Level-10
, A-Varargs
tP:
author:AuthorUsername
e.g., author:johnDoe
We expect the PR peer-review to be mutually beneficial to the reviewer and the author. i.e., you receive suggestions on how to improve your code, and get to learn alternative designs by reading others' code.
Admin Using GitHub → Guidelines for Reviewing PRs
Additional guidelines:
Comment
(i.e., not Approve
or Request changes
) Use the iP Code Dashboard to view others' code: You can now easily view iP code written by classmates, using the Links → iP Code Dashboard
item in the top navigation menu. Click on the </> icon corresponding to a student name to see the code written by that person. We encourage you to read others’ code and learn from them. If you adopt solutions from others (also encouraged), please follow our reuse policy.
Admin Module Policies → Policy on Reuse
A-Gradle
, A-CheckStyle
, Level-10
, A-Varargs
master
branch only when ready (if you push incomplete features into the master
branch, it can confuse others reviewing your PR).
A-Gradle
: Use Gradle Use Gradle to automate some of the build tasks of the project. Refer to the Gradle tutorial at the Duke repo (i.e., the repo you forked from) to find how to set up Gradle for your project.
A-CheckStyle
: Use CheckStyle optionalUse checkStyle to detect coding style violations.
If you are using Gradle for your project, refer to the Gradle tutorial at the Duke repo (i.e., the repo you forked from) to find how to use CheckStyle via Gradle. Alternatively, you can try the CheckStyle plugin for Intellij.
Extension A-Gradle
Use Gradle to automate some of the build tasks of the project. Refer to the Gradle tutorial at the Duke repo (i.e., the repo you forked from) to find how to set up Gradle for your project.
Given below are some guidance on the recommended progress at this point of the project
Admin tP: Expectations
Admin tP: Constraints
Admin tP: Grading → Criteria Used for Grading the Product Design