MergeBaseCommit was spelled incorrectly in CompareResult causing Octokit to pass
"null" no matter what the comparison is. This commit makes MergedBaseCommit (the
previous spelling) obsolete, introduces MergeBaseCommit property and adds a test
to ensure that MergeBaseCommit is set properly in CompareResult.
In the test, I specifically compared a commit with itself since the test only
cares whether the property is filled correctly or not, it is not testing whether
the internal computation is correct or not.
Closes#1258.
* Added ApiOptions overloads for RepositoryCommitClient methods
* Added overloads to Reactive clients
* Integration tests in progress
* Integration test addition in progress
* More integration tests added
* Added Ensure things
* Minor changes
* Minor changes
* Few changes
* Tried to fix errors
* Tried to fix errors
* Fixed a few things
* Fixed integration tests
* Tidying up
* Added public keyword
* Added unit tests for GetAll() in normal and reactive methods
* Minor changes
* Changed the class name
* Fixed the unit test
I was trying to create a repository and I wasn't sure which parameters
were required. Following our philosophy of exposing required parameters
in the constructor, I change the `NewRepository` object to take in a
repository name and to make that property readonly.
aedampir@gmail.com