Paul Tan

@nianfei97 > **2) Defensive programming against `null`** > > There appears to be two different ways to prevent null values from being passed, namely `requireNonNull(arg)` and `assert arg != null`....

> Other than the fact that the corrections can be automated (which is why se-edu developers rarely have the checkstyle problems that you listed wink), having these stringent style checks...

@tenvinc > Then, just for curiosity sake, why the design choice to throw an exception instead of using the assert keyword provided by Java? I mentioned the reason in https://github.com/se-edu/addressbook-level4/issues/931#issuecomment-438366556...

@hidingmode > For my project, I had to add a new component for certain (but not all) command classes in Logic to call. Judging by how Model (existing dependency of...

My mistake for #265 , I let the PR go way off tangent. Any PR for this issue should just allow *comma* or *hyphen* **only**. If people want other characters...

@MightyCupcakes > I propose moving all user-related messages to the Messages class with each command having their own Map of various common messages to their respective strings. Consistency is good,...

OK, just make sure that the real underlying root causes and areas for improvement have been identified in this issue :stuck_out_tongue: If unsure, it'll be better to do things in...

> To that end, I'm interested to know how much work it is to do a _minimal_ morph of AB3.5 from `Person` to `Task` with full use of IDE automation....