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

[Kwong Chung Yue Jerry] Duke Increments #354

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
60 commits
Select commit Hold shift + click to select a range
65f72a8
Add support for Gradle workflow
j-lum Aug 6, 2019
0112efe
Add sample checkstyle configuration
j-lum Aug 12, 2019
cfd6da7
Change file mode on `gradle` to be executable
j-lum Aug 18, 2019
6e6ace1
Merge pull request #12 from j-lum/gradle+x
j-lum Aug 18, 2019
7a4a326
level-1
Aug 20, 2019
a8142cf
level-2
Aug 20, 2019
0f80053
level-3
Aug 20, 2019
a2b0ce6
level - 4
Aug 21, 2019
e8bcbc1
level - 5
Aug 22, 2019
06e54ae
level - 6
Aug 22, 2019
e4518be
level-7
Aug 26, 2019
5a363f9
level-7.1
Aug 27, 2019
c12d10f
level-8
Aug 27, 2019
b4f0619
Merge branch 'branch-level-8' into branch-level-7
Aug 27, 2019
3ae8135
merge with master
Aug 27, 2019
b598dce
more OOP
Aug 28, 2019
69dcfeb
more OOP
Aug 28, 2019
92f2560
JUnit Testing
Aug 28, 2019
b2e96ca
JUnit testing fix 1
Aug 28, 2019
06656d4
no message
Aug 28, 2019
e129cef
no message
Aug 28, 2019
5858c5c
level-JavaDoc
Aug 28, 2019
4b25475
level-9
Aug 28, 2019
9d83ec7
Merge branch 'branch-level-9'
Aug 28, 2019
c347dcd
no message
Aug 28, 2019
1930023
level-9.1
Aug 28, 2019
e24527d
level-9.2
Aug 28, 2019
0e13863
level-9.3
Aug 28, 2019
981a5b1
Merge branch 'gradle'
Aug 30, 2019
df314ce
A-Gradle
Sep 4, 2019
092d51f
A-Gradle
Sep 4, 2019
4e1f3d4
A-gradle
Sep 5, 2019
483de67
Merge pull request #2 from jerryk1997/A-Gradle
jerryk1997 Sep 5, 2019
53a9de5
Level-10
Sep 10, 2019
3999aba
Level-10
Sep 10, 2019
0d7daba
no message
Sep 11, 2019
cd36226
no message
Sep 13, 2019
a0d7621
no message
Sep 17, 2019
5533e7b
Merge pull request #4 from jerryk1997/A-Assertions
jerryk1997 Sep 17, 2019
0f0a057
Changes to improve code quality (#5)
jerryk1997 Sep 17, 2019
c9e7b3e
Added way for user to specify priorities of tasks
Sep 17, 2019
e748cdb
Merge pull request #6 from jerryk1997/C-Priority
jerryk1997 Sep 17, 2019
8840412
no message
Sep 28, 2019
975801f
no message
Sep 28, 2019
7359f03
no message
Sep 28, 2019
c3e6085
no message
Sep 28, 2019
a376a11
no message
Sep 28, 2019
676b00d
no message
Sep 28, 2019
8b248a0
no message
Sep 28, 2019
8d9d8e8
no message
Sep 28, 2019
b0e6354
no message
Sep 28, 2019
cd1d5f6
Editing the README.md
Sep 28, 2019
2410c7a
Updating the User Guide
Sep 28, 2019
21b28fd
no message
Sep 28, 2019
8833dab
Cross platform compatibility added
Sep 28, 2019
8c76a95
Added and ran checkstyle
Sep 29, 2019
73fa35f
no message
Sep 29, 2019
d9f432d
Updated java docs
Sep 29, 2019
b6c9579
no message
Sep 29, 2019
c122a3c
Updated user guide
Sep 29, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/main/java/Ui.java
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,11 @@ public void showTaskDeleted(Task task, TaskList taskList) {
printLine();
}

/**
* Prints the list of tasks which has descriptions
* matching that of a target String.
* @param taskList List of matching tasks
*/
public void showFoundTasks(LinkedList<Task> taskList) {
ListIterator<Task> iter = taskList.listIterator();
Task current;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Your code looks pretty clean with the accurate variable naming as well as javadocs comments. Perhaps you could consider making packages to make it more organized? Otherwise it looks good

Expand Down