184 Update UG #190
Merged
184 Update UG #190
Conversation
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
✅ Deploy Preview for tp-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov Report
@@ Coverage Diff @@
## master #190 +/- ##
============================================
- Coverage 56.60% 56.52% -0.09%
- Complexity 648 649 +1
============================================
Files 118 118
Lines 2959 2967 +8
Branches 362 362
============================================
+ Hits 1675 1677 +2
- Misses 1068 1071 +3
- Partials 216 219 +3
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
added 15 commits
October 30, 2023 19:58
…e-ug * 'master' of https://github.com/zannloo/tp: (33 commits) Fix test cases Fix checkstyle error Allow create task to allStu for root directory Refactor to use enum Category Create Category enum Check duplicate task for bulk task assignment Check duplicate task for all children Refactor test cases Email should be lower case Add more checks Add isAnyFieldEdited missing check Make it case insensitive Edit message Fix bug in creating new entity Add method to get entity with given id Edit message format Add methods to check existance of id in ProfBook Add test cases Fix bug in constructor with empty data Add constructors with empty model data ...
…e-ug * 'master' of https://github.com/zannloo/tp: (29 commits) Add help option Fix checkstyle issue Fix checkstyle issue Add test for EditCommandTest class Add test for EditCommandTest class Update name of test Add test case for DeleteForStudentsAndGroupsCommandTest class Add test case for CreateGroupCommandTest class Add test case for CreateGroupCommandTest class Add test case to CreateDeadlineCommandTest class Add test case to CreateTodoCommandTest class Edit error message Fix test cases Fix checkstyle error Check options are valid before tokenize Check if all options given are valid Fix bug Add util to check options in argString are valid Add methods for option checking Add test to test toString methods ...
…e-ug * 'master' of https://github.com/zannloo/tp: (34 commits) Refactor with new interface Refactor with new ReadOnlyTaskList Implement IChildrenManager Implement ITaskListManager Refactor with new ReadOnlyTaskList Refactor with new interfaces Change ReadOnlyTaskList to interface Add interface for ChildrenManager Add interface for TaskListManager Change set to list Sort before getAllChildren Implement Comparator Update message usage Update message usage Update message usage Update message usage Update message usage for every command Standardise message usage for every command Remove test case Standardise the message usage for every command ...
NereusWB922
approved these changes
Nov 2, 2023
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.