- Do any leftover iP tasks from the previous week
- Create a PR to the upstream repo
- Add Increments as parallel branches:
Level-7
,Level-8
- Add Increments:
A-MoreOOP
,A-Packages
,A-JUnit
,A-Jar
- Add Increments as parallel branches:
A-JavaDoc
,A-CodingStandard
,Level-9
Reminder about the deadline for the weekly project tasks:
1 Do any leftover iP tasks from the previous week
2 Create a PR to the upstream repo
- Create a pull request (PR) from your fork to the upstream repo. Note the following:
- Create the PR from the
master
branch of your fork to themaster
branch of the upstream repo (https://github.com/nus-cs2103-AY2021S2/ip) - Set the PR name as
[{Your name}] iP
e.g.,[John Doe] iP
If you are reluctant to give full name, you may give the first half of your name only.
You may leave the description as empty. - If you created the PR correctly, it should appear in the list of PRs here.
- Steps for creating a PR is given in this textbook topic (steps 5 onwards):
- Create the PR from the
The PR will update automatically to reflect your latest code every time you push code to your fork. As a result, it provides a convenient way for us to access the current state of all your iP code from one location.
3 Add Increments as parallel branches: Level-7
, Level-8
- Do Level 7 in a branch named
branch-Level-7
. Without merging that branch, go back to themaster
branch and implement the other increment in a separate branch named similar to the first (i.e.,branch-{increment ID}
). Now, go back to themaster
branch and merge the two branches one after the other. As before, tag the commit (in themaster
branch, after merging) that achieves the respective deliverable, and push to your fork.
Remember to push the branches to your fork so that the bot can detect them.
Only merged branches are detected by the script. After merging a branchb1
to themaster
branch, you need to push both themaster
and theb1
branches to the fork. Pushing themaster
branch does not automatically take theb1
branch along with it just because it is already merged to themaster
branch.
Advanced git users: do not delete the branch after merging.
Merge without a fast-forward so that git creates a separate commit for the merge.
4 Add Increments: A-MoreOOP
, A-Packages
, A-JUnit
, A-Jar
Why more OOP?
One the increments below asks you to push the design more towards the OOP approach. This is a good point to remind you that OOP is not a silver bullet or always the right choice. While it is the most popular and possibly the right choice for majority of situations, it has plenty of detractors and known problems.
OOP has been chosen as the primary paradigm for this module and you are expected to try to push it to its limits. That should give you a first-hand experience of OOP's strengths and weaknesses. Furthermore, OOP (or any other paradigm) will appear worse than it really is if not used correctly, and learning to use it correctly in increasingly larger systems is another objective you can aim for in this module. As you do the tP later, you'll also realize that while OOP is used for the internal design of its components, the higher-level design is not specifically an OOP one.
We do not prohibit the use of other paradigms, however. For example, if you find a place where the functional approach is better, go ahead and use it. As you know, Java supports functional programming to a certain extent. In fact, the tP code given to you uses small snippets of functional-style code in several places.
- As in the previous week, commit, tag, and push, as you do the following increments in the
master
branch (no need to use separate branches).
5 Add Increments as parallel branches: A-JavaDoc
, A-CodingStandard
, Level-9
- As in the step 1 above, implement these three increments as three parallel branches first (branch names:
branch-A-JavaDoc
,branch-A-CodingStandard
,branch-Level-9
), and then merge them one-by-one. Hopefully, you will encounter some merge conflicts so that you get to practice de-conflicting branches. - The Java and Git standards to follow (for
A-CodingStandard
) are given in this page.