-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add new exceptions #36
Conversation
@@ -20,7 +20,7 @@ public void addTravelActivity(TravelActivity travelActivity){ | |||
travelActivities.add(travelActivity); | |||
noOfTasks += 1; | |||
int newSize = noOfTasks; | |||
assert newSize != initialListSize + 1 : "There is an error with list size!"; | |||
assert newSize == initialListSize + 1 : "There is an error with list size!"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job in catching this assertion bug
public static void main(String[] args) { | ||
new Duke().runBot(); | ||
Ui.printGreeting(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job in cleaning up the code
@@ -35,6 +37,16 @@ public static void printLine(){ | |||
System.out.println("____________________________________________________________"); | |||
} | |||
|
|||
public static void printException(OmniException exception){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
87ed3d1
into
AY2324S2-CS2113-T12-4:master
No description provided.