-
Notifications
You must be signed in to change notification settings - Fork 54
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
Made constructors more defensive for Course Package #89
base: master
Are you sure you want to change the base?
Conversation
Hi @monikernemo, your pull request title is invalid. For PR sent as submission of learning a topic/graded exercise, the PR name should be in the format of For team PR, the PR name should be in the format of 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. |
…entation Update documentation
shifted a level check to gradtrak
This reverts commit 1bf985e
Update the tests for remall and others. Closes nus-cs2103-AY1819S2#77
Updating Junit Tests
added test case for codecontainskeywordpredicate
add mods to a level regex
Pushing test cases to the limit
updated geq regex
Made constructor classes in Course more defensive
Also added some test cases for ContainsAll, ContainsAtLeast and CourseRequirement