* Add support for /mets/public_keys/<keyType>
* "files.insertFinalNewline": false
* revert and make setttings.json change csharp only
* formatting
* remove final new line
---------
Co-authored-by: Nick Floyd <139819+nickfloyd@users.noreply.github.com>
* reworks all number parameter names to represent what they actually are. Refactors some types to be the appropriate types based on OpenAPI and docs.
* updates interfaces and implementations for id naming
* updates reactive to match sync SDKs
* Fixes ids for Releases, Collaborators, and Contributors
* updates the interface for releases
* update the obverable release client
* updates ids from int to long based on GH database schema
* converts a test condition to use the proper type
* updates generated paging and observable classes
* Create a GitHub App from a manifest
* Add missing InstallationPermissions
* observable and tests
* Remove ManualRoute on Observable route
---------
Co-authored-by: Nick Floyd <139819+nickfloyd@users.noreply.github.com>
feat: added Missing APIOption overload for PackagesClient.GetAll* #2923
added missing APIOption overload for PackagesClient and ObservablePackagesClient
added overload for optional parameter packageVisibility to be a nonbreaking change
extended PackagesClientTests.cs to be conform with RepositoriesClientTests.cs to take ApiOptions into account
* #2927: comment id model update to long instead of int
* #2927: code review fixes (1)
* #2927: code review fixes (2)
* #2927: comment id model update to long instead of int: unit tests fix
* #2927: code review fixes
* Fixed most names of parameters
---------
Co-authored-by: Victor Vorobyev <victor@myrtle-sa.com>
Co-authored-by: Brian C. Arnold <brian.arnold@spiderrock.net>
* fix: added `RequiredSignatures` to the `BranchProtectionSettingsUpdate` types.
- Fixes#2855
- Updated `BranchProtectionSettingsUpdate` by adding `RequiredSignatures`.
- Update all integration `xUnit` tests to check for the `RequiredSignatures` property.
* docs: add comment documenting the `RequiredSignatures` property
* initial tests and implementation of Copilot for Business client API
* updated billing settings documentation
* renames and refactors - clarity and simplified
* using context to ensure license clean up
* extra documentation and used ApiOptions instead of custom class
* implemented observable clients
* Fixing convention issues
* renaming for clarity
---------
Co-authored-by: Nick Floyd <139819+nickfloyd@users.noreply.github.com>
* Fix handling for Streams
Fix `NullReferenceException` if raw content was handled as a string rather than a stream by `HttpClientAdapter.BuildResponse()`.
Resolves#2789.
* Add Connection.GetRaw tests
Add tests for `Connection.GetRaw()` and for #2789.
---------
Co-authored-by: Keegan Campbell <me@kfcampbell.com>
* (GH-2802) Add unit test to demonstrate the problem
This unit test currently fails which demonstrates the problem and it should pass when I am done with the PR
* (GH-2802) Add a GetRaw method that accepts a timeout
I debated adding an optional parameter for the timeout to the existing GetRaw method (which would be my personal preference) but it would be a breaking change and I doubt the Octokit team would be interested in such a change
* (GH-2802) Invoke `GetRaw` rather than `Get>byte[]>` when retrieving a repo's archive content.
This ensure stream content are handled properly and solves the problem described in GitHub issue 2802
* (GH-2802) Fix unit tests that got broken due to my recent change to the GetArchive method
* (GH-2802) Fix formatting
* (GH-2802) Fix more formatting
