se-edu / addressbook-level4 / 3965

Builds Branch Commit Type Ran Committer Via Coverage
3965 master UndoableCommand: Update redo() to restore previous view before executing The method undo() sets the Addressbook view to show all persons after restoring the Addressbook state. This leads to redo() potentially executing its command on the incorre... Pull #792 web-flow travis-ci pending completion   set done
3964 master Add documentation on how to set up and use Coveralls There is no documentation on how to set up Coveralls. With a detailed explanation, new developers can set up Coveralls easily and receive coverage report after each Travis build. Let's add d... Pull #787 web-flow travis-ci pending completion   set done
3963 master EditCommand: Store `Person` to be edited EditCommand only stores the `Index` of the `Person` in the current `filteredPersonList` to be edited. This could cause the wrong `Person` to be edited during the re-execution of the same command during `r... Pull #792 web-flow travis-ci pending completion   set done
3962 master Person: Remove copy constructor This copy constructor existed for the purpose of defensive copying. Person is immutable. There is no need for defensive copying. Let’s remove it. Some methods depended on Person's copy constructor to throw NullP... Pull #765 web-flow travis-ci pending completion   set done
3961 master Person: Remove copy constructor This copy constructor existed for the purpose of defensive copying. Person is immutable. There is no need for defensive copying. Let’s remove it. Some methods depended on Person's copy constructor to throw NullP... Pull #765 web-flow travis-ci pending completion   set done
3960 master Model: Add deleteTag(Tag) The Model API does not expose a way to delete a particular Tag entirely from the AddressBook (i.e. removing the Tag from all Persons and purging the Tag in the AddressBook's tag list). Let's add Model#deleteTag(Tag). Pull #790 web-flow travis-ci pending completion   set done
3959 master Model: Add deleteTag(Tag) The Model API does not expose a way to delete a particular Tag entirely from the AddressBook (i.e. removing the Tag from all Persons and purging the Tag in the AddressBook's tag list). Let's add Model#deleteTag(Tag). Pull #790 web-flow travis-ci pending completion   set done
3958 master Add documentation on how to set up and use Coveralls There is no documentation on how to set up Coveralls. With a detailed explanation, new developers can set up Coveralls easily and receive coverage report after each Travis build. Let's add d... Pull #787 web-flow travis-ci pending completion   set done
3957 master Add documentation on how to set up and use Coveralls There is no documentation on how to set up Coveralls. With a detailed explanation, new developers can set up Coveralls easily and receive coverage report after each Travis build. Let's add d... Pull #787 web-flow travis-ci pending completion   set done
3956 master Add documentation on how to set up and use Coveralls There is no documentation on how to set up Coveralls. With a detailed explanation, new developers can set up Coveralls easily and receive coverage report after each Travis build. Let's add d... Pull #787 web-flow travis-ci pending completion   set done
3955 master EditPersonDescriptor#getTags(): Return unmodifiable set getTags() returns the same set of tags store by this object. Callers can modify the state of the Set<Tag> externally, thus breaking EditPersonDescriptor’s encapsulation. Let’s teach getTag... Pull #795 web-flow travis-ci pending completion   set done
3954 master AddressBook: Inline setter methods setPersons(...) and setTags(...) are only used once. It is unnecessary have the logic of setting persons and tags to be extracted as separate setter methods, since they do not help to introduce a new level of a... Pull #794 web-flow travis-ci pending completion   set done
3953 master Add documentation on how to set up and use Coveralls There is no documentation on how to set up Coveralls. With a detailed explanation, new developers can set up Coveralls easily and receive coverage report after each Travis build. Let's add d... Pull #787 web-flow travis-ci pending completion   set done
3952 master Update BrowserPanel tests to pass if Google redirects webpage Google rejects search requests if it detects unusually high traffic, by redirecting us another web page to verify that we are not robots. This may occur when too many students are usin... Pull #793 web-flow travis-ci pending completion   set done
3951 master Update BrowserPanel tests to pass if Google redirects webpage Google rejects search requests if it detects unusually high traffic, by redirecting us another web page to verify that we are not robots. This may occur when too many students are usin... Pull #793 web-flow travis-ci pending completion   set done