Skip to content
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

[Taha Jailani] Duke Increments #371

Open
wants to merge 50 commits into
base: master
Choose a base branch
from

Conversation

PhireHandy
Copy link

No description provided.

@nus-se-pr-bot
Copy link

Hi @PhireHandy, your pull request title is invalid.

For PR sent as submission of iP (Individual Project), the PR name should be in the format of [{Your name}] Duke Increments (such as [John Doe] Duke Increments).

Please follow the above format strictly and edit your title for reprocessing.

Note: this comment is posted by a bot. If you believe this is done in error, please create an issue at nus-se-pr-bot and add a link to this PR.

@damithc
Copy link
Collaborator

damithc commented Sep 7, 2019

Please rectify the PR title to match the requirement. The PR author can edit the PR title.

@PhireHandy PhireHandy changed the title [{Taha Jailani}] Duke Increments [Taha Jailani] Duke Increments Sep 14, 2019
PhireHandy and others added 21 commits September 24, 2019 14:42
# Conflicts:
#	src/main/java/Duke.java
#	src/main/java/command/DeadlineCommand.java
#	src/main/java/command/DeleteCommand.java
#	src/main/java/command/DoneCommand.java
#	src/main/java/command/EventCommand.java
#	src/main/java/exception/DukeException.java
#	src/main/java/main/Storage.java
#	src/main/java/main/UI.java
#	src/main/java/task/Event.java
#	src/main/java/task/Todo.java
# Conflicts:
#	src/main/java/main/UI.java
#	src/main/java/task/Task.java
To be implemented:
Full support of all Duke's previous features with the new GUI
…s in Main directory follow the CheckStyle. (#5)

Assertion checks to ensure that the correct types of command are passed as inputs.
Please note that all refactoring had already been done prior to this commit, incrementally.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants