-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fix undo bug for find and view #219
Conversation
- need to fix 3 failed test cases - clist cannot undo yet
…giang # Conflicts: # src/main/java/seedu/address/logic/commands/contact/CViewCommand.java
Codecov Report
@@ Coverage Diff @@
## master #219 +/- ##
============================================
+ Coverage 68.89% 68.94% +0.04%
- Complexity 982 1004 +22
============================================
Files 125 129 +4
Lines 3302 3355 +53
Branches 432 433 +1
============================================
+ Hits 2275 2313 +38
- Misses 843 857 +14
- Partials 184 185 +1
Continue to review full report at Codecov.
|
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.
Good work. All commands seem to work with undo/redo beside mark and link, which is not the issue of undo/redo. LGTM
No description provided.