se-edu / addressbook-level4 / 3948

Builds Branch Commit Type Ran Committer Via Coverage
3948 master BrowserPanelHandle: Update to pass test if webpage failed to load If Google rejects search requests made by our automated tests on Travis CI or AppVeyor, these tests will fail. Developers will have to re-run the build until Google stops rejecting... Pull #793 web-flow travis-ci pending completion   set done
3947 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
3946 master [#415] Ui: fix default PersonListPanel background color (#791) Normally, the background of the PersonListPanel is gray in color. However, it changes to white color when no Person is listed. This does not fit the dark theme style of other part... push Zhiyuan-Amos travis-ci pending completion   set done
3945 master Ui: fix default PersonListPanel background color Normally, the background of the PersonListPanel is gray in color. However, it changes to white color when no Person is listed. This does not fit the dark theme style of other parts of the Ui as th... Pull #791 web-flow travis-ci pending completion   set done
3944 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
3943 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
3942 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. As the copy constructor throws NullPointerException if the given ... Pull #765 web-flow travis-ci pending completion   set done
3940 master Ui: fix default PersonListPanel background color Normally, the background of the PersonListPanel is gray in color. However, the PersonListPanel background becomes white when no Person is listed. This does not fit the dark theme style of Ui as th... Pull #791 web-flow travis-ci pending completion   set done
3939 master UserGuide: Add command aliases Pull #785 web-flow travis-ci pending completion   set done
3938 master [#264] Replace use of `assert false` with `throw new AssertionError()` (#788) In some places of the code, we use `assert false` to indicate the execution has reached an unacceptable state. This is not compliant with our code style (as revised... push web-flow travis-ci pending completion   set done
3937 master Replace use of `assert false` with `throw new AssertionError()` In some places of the code we use `assert false` to indicate the execution has reached an unacceptable state. This is not compliant with our code style (as revised by [1]) which st... Pull #788 web-flow travis-ci pending completion   set done
3936 master Replace use of `assert false` with `throw new AssertionError()` In some places of the code we use `assert false` to indicate the execution has reached an unacceptable state. This is not compliant with our code style (as revised by [1]) which st... Pull #788 web-flow travis-ci pending completion   set done
3935 master Replace use of `assert false` with `throw new AssertionError()` There has been a code policy change[1] where it was determined that `throw new AssertionError()` could likely be better used in places where we are currently using `assert false` to ... Pull #788 web-flow travis-ci pending completion   set done
3934 master DeveloperGuide: Add scenarios to suggested programming tasks The suggested programming tasks are just a list of tasks to do. Providing the background context of the tasks makes them more interesting and real, which will give a better motivation ... Pull #789 web-flow travis-ci pending completion   set done
3933 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