* Fix whitespace/formatting with /FormatCode build option
* Update release notes
* fix a few failing integration tests
* Adjust required fields on UpdateCheckRun and NewCheckRun request models and fix tests
Tidy up field accessors and XmlDoc comments
* Update date in ReleaseNotes
* Keeping request models simple (avoid inheritance) - makes it easier when we move to generated models
* run FormatCode build task to fix whitespace/formatting
* correctly flag some GitHub Enterprise tests
* Add Release Notes
* tweak appveyor to not build branch in the repo, when a PR exists already
* travis to only build master branch and PRs
* output actual "dotnet run" command being executed for cake.frosting builds
* update latest Cake.Frosting
* use normal verbosity at the moment due to apparent conflict with "verbose" and latest SDK on appveyor
* try using double dash so dotnet executable doesnt look at --verbosity argument
* travis OSX couldn't download SDK 2.0.3 anymore, lets try the latest 2.1.300
* Add release notes
* Fix failing integration test
* Run FormatCode build task to tidy up whitespace
* Remove "UnkownUser" from release notes contributors
* Fix integration test - protected branch status can now only be retrieved by an authorized user
* Fix integration test - labels in octokit.net repo were renamed
* Run build task "FormatCode"
* Update release notes for v0.27
* Add release notes and bump version to 0.24
* run "build FormatCode" to fix up whitespace/formatting issues
* Fix failing Ssh key tests due to "validation exception". This key must be in use on github (under another user, most likely from these tests failing). Changed to a new SSH key and tweaked tests to reduce chance of a key being created and not destroyed
* Assignee and Assignees cant both be specified on NewIssue. We missed this one in the PR. Marked Assignee as [Obsolete] and fixed tests to use Assignees
* Fix a couple of Reactions tests that were calling the wrong client methods
* Fix timeline tests - looks like the response class has changed shape a bit, it now has an Issue object in the payload and Id field isnt present (leaving Id field there in case other timeline events do use it)
* Fix some following tests that require the test user to follow more than 1 other user
* Unskip these Event tests now because apparently they work!
* add breaking changes notes
* Update ApiErrorMessageSafe to return null for empty and whitespace strings (#1540)
* return null if ApiError.Message is empty or whitespace
* Uncomment test, which now passes
* update release notes to include PR1540
* Add "Bot" AccountType, was causing a deserialization exception when running the integration test "SearchForExcludedLanguage" (#1541)
* Update to include PR1541
* add bullets to make release notes easier to read
* markup additional code mentions in notes
* Fix grammar
fields => field
A recent PR added CommitEntity but we already had
SignatureResponse expressly for this purpose.
So this commit renames SignatureResponse to Committer
and removes CommitEntity and replaces it with Committer.
We get a lot of build output because of missing XML comments that we
ignore. I'd like to stop ignoring them. To do that, we need to doc the
💩 out of everything.
Ryan Gribble