-
Notifications
You must be signed in to change notification settings - Fork 482
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
[Luah Jun Yang} iP #571
Open
LuahJunYang
wants to merge
31
commits into
nus-cs2103-AY2324S1:master
Choose a base branch
from
LuahJunYang:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[Luah Jun Yang} iP #571
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
28ad2b8
Add Gradle support
ed6d4d2
Bump gradle and lib version
Eclipse-Dominator 8f211eb
Level 0
LuahJunYang c442a23
Finished Level 1, renamed bot to Frenchie
LuahJunYang e26ad60
Added functionality to store user input as list and print out all inp…
LuahJunYang 2826c8b
Created Task class to be used for Frenchie
LuahJunYang c163e3f
Added addTask and listTask methods and a constructor to Frenchie.java
LuahJunYang 6880516
Added mark as incomplete method to Task.java and complete and uncompl…
LuahJunYang 9502380
Created new Task subclasses
LuahJunYang 6403c3d
Added if-else statements and updated toString() methods for checking…
LuahJunYang f8faf33
Created new FrenchieException class to handle basic exceptions for no…
LuahJunYang 59151ce
Added delete function to frenchie, changed tasks attribute to ArrayLi…
LuahJunYang 76c74cf
Added functions saveTasksToFile and readTasksFromFile
LuahJunYang efa2cac
Added LocalDateTime and DateTimeFormatter functionalities to construc…
LuahJunYang cf2306e
Implemented enum for Commands, abstracted UI, Parser, Storage, and Ta…
LuahJunYang 3a10e8d
Bug fixing for bye command, causing an illegal state exception when t…
LuahJunYang c59c708
Packaged all classes into package.frenchie
LuahJunYang 1435a48
Merge branch 'add-gradle-support' for 'A-Gradle'
LuahJunYang fb9f638
Updated build.gradle file, wrote J-unit tests ToDoTest and TaskTest
LuahJunYang 2ac3943
Added JavaDoc comments for Task, ToDo, Deadline, Event, Ui, Parser cl…
LuahJunYang e8b0d86
Added findMessage() method to Ui.java, find enum to Command, and retu…
LuahJunYang eae69dd
Resolved merge conflict in Ui.java
LuahJunYang 04310fd
Edited void functions to return String for the GUI. Implemented GUI s…
LuahJunYang 68fc1cb
Improving on Code Quality
LuahJunYang 4364715
Merge pull request #3 from LuahJunYang/branch-A-CodeQuality
LuahJunYang c59a6cf
Add help function
LuahJunYang 43a2dd1
Merge pull request #4 from LuahJunYang/branch-C-Help
LuahJunYang 4806d6d
Added User Guide in README.md
LuahJunYang 8633cba
Updating README for github page
LuahJunYang 3f79e97
Added Screenshot of UI
LuahJunYang 7b1615e
Updated mainClass in build.gradle to fix the jar file not running
LuahJunYang File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
As the number of keywords increase, perhaps you might want to start looking at enums to have a collection of your keywords!
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.
Seconded. In addition, perhaps you may wish to divide all the logic in your
main
method into several other methods, as the complexity of your program increases.