-
Notifications
You must be signed in to change notification settings - Fork 315
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
jerryk1997
wants to merge
60
commits into
nus-cs2103-AY1920S1:master
Choose a base branch
from
jerryk1997: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
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 0112efe
Add sample checkstyle configuration
j-lum cfd6da7
Change file mode on `gradle` to be executable
j-lum 6e6ace1
Merge pull request #12 from j-lum/gradle+x
j-lum 7a4a326
level-1
a8142cf
level-2
0f80053
level-3
a2b0ce6
level - 4
e8bcbc1
level - 5
06e54ae
level - 6
e4518be
level-7
5a363f9
level-7.1
c12d10f
level-8
b4f0619
Merge branch 'branch-level-8' into branch-level-7
3ae8135
merge with master
b598dce
more OOP
69dcfeb
more OOP
92f2560
JUnit Testing
b2e96ca
JUnit testing fix 1
06656d4
no message
e129cef
no message
5858c5c
level-JavaDoc
4b25475
level-9
9d83ec7
Merge branch 'branch-level-9'
c347dcd
no message
1930023
level-9.1
e24527d
level-9.2
0e13863
level-9.3
981a5b1
Merge branch 'gradle'
df314ce
A-Gradle
092d51f
A-Gradle
4e1f3d4
A-gradle
483de67
Merge pull request #2 from jerryk1997/A-Gradle
jerryk1997 53a9de5
Level-10
3999aba
Level-10
0d7daba
no message
cd36226
no message
a0d7621
no message
5533e7b
Merge pull request #4 from jerryk1997/A-Assertions
jerryk1997 0f0a057
Changes to improve code quality (#5)
jerryk1997 c9e7b3e
Added way for user to specify priorities of tasks
e748cdb
Merge pull request #6 from jerryk1997/C-Priority
jerryk1997 8840412
no message
975801f
no message
7359f03
no message
c3e6085
no message
a376a11
no message
676b00d
no message
8b248a0
no message
8d9d8e8
no message
b0e6354
no message
cd1d5f6
Editing the README.md
2410c7a
Updating the User Guide
21b28fd
no message
8833dab
Cross platform compatibility added
8c76a95
Added and ran checkstyle
73fa35f
no message
d9f432d
Updated java docs
b6c9579
no message
c122a3c
Updated user guide
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.
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