From 59c46a31b522f6eec3912f7f6bf8863cda9c8caa Mon Sep 17 00:00:00 2001 From: WyriHaximus <147145+WyriHaximus@users.noreply.github.com> Date: Mon, 2 Dec 2024 17:05:10 +0000 Subject: [PATCH] [GitHub] Update to 1.1.4-b119f75f25b0d5bbe0f8fa77c66a5d24 from 1.1.4-315e5678f61e5340f76bae356b2788b6 Detected Schema changes: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ``` └─┬Paths ├─┬/orgs/{org}/hooks │ └─┬GET │ └──[🔀] description (9518:20) ├─┬/repos/{owner}/{repo}/hooks/{hook_id} │ └─┬DELETE │ └──[➖] description (31044:20) └─┬/orgs/{org}/hooks/{hook_id} └─┬DELETE └──[🔀] description (9780:20) ``` | Document Element | Total Changes | Breaking Changes | |------------------|---------------|------------------| | paths | 3 | 0 | Date: 12/02/24 | Commit: New: etc/specs/GitHub/previous.spec.yaml, Original: etc/specs/GitHub/current.spec.yaml - **Total Changes**: _3_ - **Modifications**: _2_ - **Removals**: _1_ --- .../GitHub/etc/openapi-client-generator.state | 42 +++++++++---------- .../Internal/Operator/Issues/AddSubIssue.php | 1 + .../Operator/Issues/ListSubIssues.php | 2 +- .../Operator/Issues/ListSubIssuesListing.php | 2 +- .../Operator/Issues/RemoveSubIssue.php | 1 + .../Operator/Issues/ReprioritizeSubIssue.php | 1 + .../src/Internal/Router/Delete/Issues.php | 1 + .../src/Internal/Router/Delete/Seven.php | 1 - .../GitHub/src/Internal/Router/Get/Issues.php | 2 +- .../GitHub/src/Internal/Router/Get/Seven.php | 2 +- .../src/Internal/Router/List/Issues.php | 2 +- .../GitHub/src/Internal/Router/List/Seven.php | 2 +- .../src/Internal/Router/Patch/Eight.php | 1 - .../src/Internal/Router/Patch/Issues.php | 1 + .../src/Internal/Router/Post/Issues.php | 1 + .../GitHub/src/Internal/Router/Post/Seven.php | 2 +- clients/GitHub/src/Operation/Issues.php | 7 +++- .../src/PHPStan/ClientCallReturnTypes.php | 10 ++--- .../tests/Types/ClientCallReturnTypes.php | 10 ++--- etc/specs/GitHub/current.spec.yaml | 13 ++++-- 20 files changed, 59 insertions(+), 45 deletions(-) diff --git a/clients/GitHub/etc/openapi-client-generator.state b/clients/GitHub/etc/openapi-client-generator.state index bba6a6697ae..9e99116118c 100644 --- a/clients/GitHub/etc/openapi-client-generator.state +++ b/clients/GitHub/etc/openapi-client-generator.state @@ -1,5 +1,5 @@ { - "specHash": "315e5678f61e5340f76bae356b2788b6", + "specHash": "b119f75f25b0d5bbe0f8fa77c66a5d24", "generatedFiles": { "files": [ { @@ -5532,7 +5532,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Operation\/SecretScanning.php", - "hash": "b253cd61e8874afe2c5394eb970bd8b6" + "hash": "9bb033435eaa98d866efe71eb1e4ddbd" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Operation\/Activity.php", @@ -5548,7 +5548,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Operation\/Issues.php", - "hash": "a564ebaae7c66ed66ba4f270099de3cf" + "hash": "3cc0b5981bff1172487d1b0ae853249a" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Operation\/Licenses.php", @@ -5596,7 +5596,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Operation\/Repos.php", - "hash": "e08ac16d8c8ddb3d082c30c06392777c" + "hash": "736deb4195c1c2f43079eab132469082" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Operation\/Billing.php", @@ -24344,7 +24344,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Get\/Issues.php", - "hash": "6f56da7cc422b24bd72624f99f8d3459" + "hash": "3eae340a9f0f93509717648f24474e04" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Get\/Licenses.php", @@ -24516,7 +24516,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Post\/Issues.php", - "hash": "b73947ce9fb2650d56d62ddd8db8a1bc" + "hash": "ad28b5e75688ddd838df5a6e08945df1" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Post\/Pulls.php", @@ -24584,7 +24584,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Patch\/Issues.php", - "hash": "bd3cd52db296723ed2645567674d6a24" + "hash": "1816d97da53df9c45558c7192147699c" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Patch\/Pulls.php", @@ -24620,7 +24620,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/List\/Issues.php", - "hash": "76fed67d0b37e9644cc2435e63964266" + "hash": "aa33b1f8d2fd0f5eee6c0af5120886b8" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/List\/Migrations.php", @@ -24744,7 +24744,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Delete\/Issues.php", - "hash": "976b81d4a4693dab65aa5d495ece36b8" + "hash": "3cac08976533c6e465ba923340c963da" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Delete\/Pulls.php", @@ -24860,7 +24860,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Get\/Seven.php", - "hash": "515d113bfcaeab39cdb9022aab604a33" + "hash": "b1ab36693fa5c4c60f402c9e0349b1fd" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Get\/Eight.php", @@ -24896,7 +24896,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Post\/Seven.php", - "hash": "d8aa4a7b4957f5e330b4fecec00b97e7" + "hash": "b619e491673f6ed980d9896a97638a0a" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Post\/Eight.php", @@ -24936,7 +24936,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Patch\/Eight.php", - "hash": "e467d3113de1a5cbd8d67398d6184f97" + "hash": "7ea13e71eca3e33658e6f765175629a3" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Patch\/Two.php", @@ -24964,7 +24964,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/List\/Seven.php", - "hash": "df521fa68f2950fed9b9789c3f847041" + "hash": "81326c9e0e34933f9df6d7e397ac8782" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/List\/Eight.php", @@ -24992,7 +24992,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Delete\/Seven.php", - "hash": "43168d2c2f251ae741817f0d9e537fe4" + "hash": "102fb21cd796a379ef4ea2a84876be10" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Router\/Delete\/Six.php", @@ -27636,11 +27636,11 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/PHPStan\/ClientCallReturnTypes.php", - "hash": "37b661017e22bbf5b31534aa9bc4b3da" + "hash": "526aac10512dc0d92689f985b41922b4" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/tests\/\/Types\/ClientCallReturnTypes.php", - "hash": "3cbfb3883582a25a77f91cadadf8ee4a" + "hash": "e92ed3755800eb5eccdcb78430551361" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/..\/etc\/phpstan-extension.neon", @@ -31048,7 +31048,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Operator\/Issues\/RemoveSubIssue.php", - "hash": "562b423ebbd7386a533ec6432e89ef57" + "hash": "ae0dd2e3831ef830d359a0392d7ec289" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/tests\/\/Internal\/Operation\/Issues\/RemoveSubIssueTest.php", @@ -31060,7 +31060,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Operator\/Issues\/ListSubIssues.php", - "hash": "c82ca46e2bbdc84d13cb00af5e2aee29" + "hash": "18a3738c19bb2c6f33a12e8a0fc324bd" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/tests\/\/Internal\/Operation\/Issues\/ListSubIssuesTest.php", @@ -31072,7 +31072,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Operator\/Issues\/ListSubIssuesListing.php", - "hash": "d45c1b4569edb45e2d2b8c21453a4318" + "hash": "b005722a6b2608dc0e35512d3f1ae729" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/tests\/\/Internal\/Operation\/Issues\/ListSubIssuesListingTest.php", @@ -31084,7 +31084,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Operator\/Issues\/AddSubIssue.php", - "hash": "f9b1f25c76aa81fba345e2d0421dfec6" + "hash": "1c1e1db4d1839f82494e8a61886fd223" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/tests\/\/Internal\/Operation\/Issues\/AddSubIssueTest.php", @@ -31096,7 +31096,7 @@ }, { "name": ".\/clients\/GitHub\/etc\/..\/\/src\/\/Internal\/Operator\/Issues\/ReprioritizeSubIssue.php", - "hash": "41244efd13e59369e569d1d295b9c306" + "hash": "987ccc2d51482942020e95f85018d9fc" }, { "name": ".\/clients\/GitHub\/etc\/..\/\/tests\/\/Internal\/Operation\/Issues\/ReprioritizeSubIssueTest.php", diff --git a/clients/GitHub/src/Internal/Operator/Issues/AddSubIssue.php b/clients/GitHub/src/Internal/Operator/Issues/AddSubIssue.php index cc91c8e3fa6..44812ca22d6 100644 --- a/clients/GitHub/src/Internal/Operator/Issues/AddSubIssue.php +++ b/clients/GitHub/src/Internal/Operator/Issues/AddSubIssue.php @@ -24,6 +24,7 @@ public function __construct(private Browser $browser, private AuthenticationInte { } + /** @return */ public function call(string $owner, string $repo, int $issueNumber, array $params): Issue { $operation = new \ApiClients\Client\GitHub\Internal\Operation\Issues\AddSubIssue($this->requestSchemaValidator, $this->responseSchemaValidator, $this->hydrator, $owner, $repo, $issueNumber); diff --git a/clients/GitHub/src/Internal/Operator/Issues/ListSubIssues.php b/clients/GitHub/src/Internal/Operator/Issues/ListSubIssues.php index 062750d6b81..cd58df65a8a 100644 --- a/clients/GitHub/src/Internal/Operator/Issues/ListSubIssues.php +++ b/clients/GitHub/src/Internal/Operator/Issues/ListSubIssues.php @@ -24,7 +24,7 @@ public function __construct(private Browser $browser, private AuthenticationInte { } - /** @return iterable */ + /** @return Observable */ public function call(string $owner, string $repo, int $issueNumber, int $perPage = 30, int $page = 1): iterable { $operation = new \ApiClients\Client\GitHub\Internal\Operation\Issues\ListSubIssues($this->responseSchemaValidator, $this->hydrator, $owner, $repo, $issueNumber, $perPage, $page); diff --git a/clients/GitHub/src/Internal/Operator/Issues/ListSubIssuesListing.php b/clients/GitHub/src/Internal/Operator/Issues/ListSubIssuesListing.php index 199121d8d86..2da621ae9f0 100644 --- a/clients/GitHub/src/Internal/Operator/Issues/ListSubIssuesListing.php +++ b/clients/GitHub/src/Internal/Operator/Issues/ListSubIssuesListing.php @@ -24,7 +24,7 @@ public function __construct(private Browser $browser, private AuthenticationInte { } - /** @return iterable */ + /** @return Observable */ public function call(string $owner, string $repo, int $issueNumber, int $perPage = 30, int $page = 1): iterable { $operation = new \ApiClients\Client\GitHub\Internal\Operation\Issues\ListSubIssuesListing($this->responseSchemaValidator, $this->hydrator, $owner, $repo, $issueNumber, $perPage, $page); diff --git a/clients/GitHub/src/Internal/Operator/Issues/RemoveSubIssue.php b/clients/GitHub/src/Internal/Operator/Issues/RemoveSubIssue.php index d67b7062c9c..429f780c076 100644 --- a/clients/GitHub/src/Internal/Operator/Issues/RemoveSubIssue.php +++ b/clients/GitHub/src/Internal/Operator/Issues/RemoveSubIssue.php @@ -24,6 +24,7 @@ public function __construct(private Browser $browser, private AuthenticationInte { } + /** @return */ public function call(string $owner, string $repo, int $issueNumber, array $params): Issue { $operation = new \ApiClients\Client\GitHub\Internal\Operation\Issues\RemoveSubIssue($this->requestSchemaValidator, $this->responseSchemaValidator, $this->hydrator, $owner, $repo, $issueNumber); diff --git a/clients/GitHub/src/Internal/Operator/Issues/ReprioritizeSubIssue.php b/clients/GitHub/src/Internal/Operator/Issues/ReprioritizeSubIssue.php index d46a25ce427..afebfc2c71f 100644 --- a/clients/GitHub/src/Internal/Operator/Issues/ReprioritizeSubIssue.php +++ b/clients/GitHub/src/Internal/Operator/Issues/ReprioritizeSubIssue.php @@ -24,6 +24,7 @@ public function __construct(private Browser $browser, private AuthenticationInte { } + /** @return */ public function call(string $owner, string $repo, int $issueNumber, array $params): Issue { $operation = new \ApiClients\Client\GitHub\Internal\Operation\Issues\ReprioritizeSubIssue($this->requestSchemaValidator, $this->responseSchemaValidator, $this->hydrator, $owner, $repo, $issueNumber); diff --git a/clients/GitHub/src/Internal/Router/Delete/Issues.php b/clients/GitHub/src/Internal/Router/Delete/Issues.php index d07c17c4684..d2b673be49e 100644 --- a/clients/GitHub/src/Internal/Router/Delete/Issues.php +++ b/clients/GitHub/src/Internal/Router/Delete/Issues.php @@ -130,6 +130,7 @@ public function unlock(array $params): WithoutBody return $operator->call($arguments['owner'], $arguments['repo'], $arguments['issue_number']); } + /** @return */ public function removeSubIssue(array $params): Issue { $arguments = []; diff --git a/clients/GitHub/src/Internal/Router/Delete/Seven.php b/clients/GitHub/src/Internal/Router/Delete/Seven.php index 20bd064d13f..4465f38696a 100644 --- a/clients/GitHub/src/Internal/Router/Delete/Seven.php +++ b/clients/GitHub/src/Internal/Router/Delete/Seven.php @@ -19,7 +19,6 @@ public function __construct(private Routers $routers) { } - /** @return |Schema\Issue */ public function call(string $call, array $params, array $pathChunks): WithoutBody|Json|\ApiClients\Client\GitHub\Schema\Operations\Codespaces\DeleteFromOrganization\Response\ApplicationJson\Accepted\Application\Json|CodeScanningAnalysisDeletion|Issue|BasicError|PullRequestSimple { if ($pathChunks[0] === '') { diff --git a/clients/GitHub/src/Internal/Router/Get/Issues.php b/clients/GitHub/src/Internal/Router/Get/Issues.php index 324e3cb0a0f..6462ce7a2d6 100644 --- a/clients/GitHub/src/Internal/Router/Get/Issues.php +++ b/clients/GitHub/src/Internal/Router/Get/Issues.php @@ -800,7 +800,7 @@ public function listLabelsOnIssue(array $params): iterable|BasicError return $operator->call($arguments['owner'], $arguments['repo'], $arguments['issue_number'], $arguments['per_page'], $arguments['page']); } - /** @return iterable */ + /** @return Observable */ public function listSubIssues(array $params): iterable { $arguments = []; diff --git a/clients/GitHub/src/Internal/Router/Get/Seven.php b/clients/GitHub/src/Internal/Router/Get/Seven.php index 37556ffbcb9..171aa5e5e13 100644 --- a/clients/GitHub/src/Internal/Router/Get/Seven.php +++ b/clients/GitHub/src/Internal/Router/Get/Seven.php @@ -63,7 +63,7 @@ public function __construct(private Routers $routers) { } - /** @return Observable|Observable||Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Schema\BasicError|Observable|Observable|Observable|Observable|Observable|Observable|Observable|iterable|Schema\ConvertedNoteToIssueIssueEvent|Schema\TimelineCommentEvent|Schema\TimelineCrossReferencedEvent|Schema\TimelineCommittedEvent|Schema\TimelineReviewedEvent|Schema\TimelineLineCommentedEvent|Schema\TimelineCommitCommentedEvent|Schema\TimelineAssignedIssueEvent|Schema\TimelineUnassignedIssueEvent|Schema\StateChangeIssueEvent>|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable */ + /** @return Observable|Observable||Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Schema\BasicError|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Schema\ConvertedNoteToIssueIssueEvent|Schema\TimelineCommentEvent|Schema\TimelineCrossReferencedEvent|Schema\TimelineCommittedEvent|Schema\TimelineReviewedEvent|Schema\TimelineLineCommentedEvent|Schema\TimelineCommitCommentedEvent|Schema\TimelineAssignedIssueEvent|Schema\TimelineUnassignedIssueEvent|Schema\StateChangeIssueEvent>|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable */ public function call(string $call, array $params, array $pathChunks): iterable|OidcCustomSub|Ok|\ApiClients\Client\GitHub\Schema\Operations\Actions\ListSelfHostedRunnersInGroupForOrg\Response\ApplicationJson\Ok|\ApiClients\Client\GitHub\Schema\Operations\Actions\ListLabelsForSelfHostedRunnerForOrg\Response\ApplicationJson\Ok|\ApiClients\Client\GitHub\Schema\Operations\Actions\ListSelectedReposForOrgSecret\Response\ApplicationJson\Ok|Json|WithoutBody|\ApiClients\Client\GitHub\Schema\Operations\Codespaces\ListSelectedReposForOrgSecret\Response\ApplicationJson\Ok\Application\Json|\ApiClients\Client\GitHub\Schema\Operations\Dependabot\ListSelectedReposForOrgSecret\Response\ApplicationJson\Ok\Application\Json|HookDelivery|TeamDiscussion|TeamMembership|TeamProject|Artifact|ActionsCacheUsageByRepository|Job|ActionsWorkflowAccessToRepository|SelectedActions|ActionsGetDefaultWorkflowPermissions|Runner|WorkflowRun|ActionsPublicKey|ActionsSecret|ActionsVariable|Workflow|BranchProtection|\ApiClients\Client\GitHub\Schema\Operations\Checks\ListForSuite\Response\ApplicationJson\Ok|CodeScanningAlert|CodeScanningAnalysis|CodeScanningSarifsStatus|CodespacesPublicKey|RepoCodespacesSecret|RepositoryCollaboratorPermission|\ApiClients\Client\GitHub\Schema\Operations\Checks\ListForRef\Response\ApplicationJson\Ok\Application\Json|\ApiClients\Client\GitHub\Schema\Operations\Checks\ListSuitesForRef\Response\ApplicationJson\Ok|CombinedCommitStatus|BasicError|DependabotAlert|DependabotPublicKey|DependabotSecret|\ApiClients\Client\GitHub\Schema\Operations\Repos\ListDeploymentBranchPolicies\Response\ApplicationJson\Ok|\ApiClients\Client\GitHub\Schema\Operations\Repos\GetAllDeploymentProtectionRules\Response\ApplicationJson\Ok|\ApiClients\Client\GitHub\Schema\Operations\Actions\ListEnvironmentSecrets\Response\ApplicationJson\Ok\Application\Json|\ApiClients\Client\GitHub\Schema\Operations\Actions\ListEnvironmentVariables\Response\ApplicationJson\Ok\Application\Json|Blob|GitCommit|GitRef|GitTag|GitTree|WebhookConfig|IssueComment|IssueEvent|PageBuild|PagesDeploymentStatus|PullRequestReviewComment|PullRequestReviewRequest|ReleaseAsset|Release|RuleSuite|SecretScanningAlert|TeamDiscussionComment|PackageVersion { if ($pathChunks[0] === '') { diff --git a/clients/GitHub/src/Internal/Router/List/Issues.php b/clients/GitHub/src/Internal/Router/List/Issues.php index 36efed10fd6..51cbca308e5 100644 --- a/clients/GitHub/src/Internal/Router/List/Issues.php +++ b/clients/GitHub/src/Internal/Router/List/Issues.php @@ -610,7 +610,7 @@ public function listLabelsOnIssueListing(array $params): iterable|BasicError } while (count($items) > 0); } - /** @return iterable */ + /** @return Observable */ public function listSubIssuesListing(array $params): iterable { $arguments = []; diff --git a/clients/GitHub/src/Internal/Router/List/Seven.php b/clients/GitHub/src/Internal/Router/List/Seven.php index 103c2d20b23..496fd87c523 100644 --- a/clients/GitHub/src/Internal/Router/List/Seven.php +++ b/clients/GitHub/src/Internal/Router/List/Seven.php @@ -14,7 +14,7 @@ public function __construct(private Routers $routers) { } - /** @return Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Schema\BasicError|Observable|Observable|Observable|Observable|iterable|Schema\ConvertedNoteToIssueIssueEvent|Schema\TimelineCommentEvent|Schema\TimelineCrossReferencedEvent|Schema\TimelineCommittedEvent|Schema\TimelineReviewedEvent|Schema\TimelineLineCommentedEvent|Schema\TimelineCommitCommentedEvent|Schema\TimelineAssignedIssueEvent|Schema\TimelineUnassignedIssueEvent|Schema\StateChangeIssueEvent>|Observable|Observable|Observable|Observable|Observable|Observable */ + /** @return Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Observable|Schema\BasicError|Observable|Observable|Observable|Observable|Observable|Schema\ConvertedNoteToIssueIssueEvent|Schema\TimelineCommentEvent|Schema\TimelineCrossReferencedEvent|Schema\TimelineCommittedEvent|Schema\TimelineReviewedEvent|Schema\TimelineLineCommentedEvent|Schema\TimelineCommitCommentedEvent|Schema\TimelineAssignedIssueEvent|Schema\TimelineUnassignedIssueEvent|Schema\StateChangeIssueEvent>|Observable|Observable|Observable|Observable|Observable|Observable */ public function call(string $call, array $params, array $pathChunks): iterable|BasicError { if ($pathChunks[0] === '') { diff --git a/clients/GitHub/src/Internal/Router/Patch/Eight.php b/clients/GitHub/src/Internal/Router/Patch/Eight.php index f90701aa124..29e8d7cc4a2 100644 --- a/clients/GitHub/src/Internal/Router/Patch/Eight.php +++ b/clients/GitHub/src/Internal/Router/Patch/Eight.php @@ -17,7 +17,6 @@ public function __construct(private Routers $routers) { } - /** @return |Schema\Issue */ public function call(string $call, array $params, array $pathChunks): ProtectedBranchPullRequestReview|StatusCheckPolicy|WithoutBody|Issue { if ($pathChunks[0] === '') { diff --git a/clients/GitHub/src/Internal/Router/Patch/Issues.php b/clients/GitHub/src/Internal/Router/Patch/Issues.php index 657352afb33..99c2b1d6b49 100644 --- a/clients/GitHub/src/Internal/Router/Patch/Issues.php +++ b/clients/GitHub/src/Internal/Router/Patch/Issues.php @@ -131,6 +131,7 @@ public function updateComment(array $params): IssueComment return $operator->call($arguments['owner'], $arguments['repo'], $arguments['comment_id'], $params); } + /** @return */ public function reprioritizeSubIssue(array $params): Issue { $arguments = []; diff --git a/clients/GitHub/src/Internal/Router/Post/Issues.php b/clients/GitHub/src/Internal/Router/Post/Issues.php index 744f55d7529..e0d9125dd12 100644 --- a/clients/GitHub/src/Internal/Router/Post/Issues.php +++ b/clients/GitHub/src/Internal/Router/Post/Issues.php @@ -168,6 +168,7 @@ public function addLabels(array $params): iterable|BasicError return $operator->call($arguments['owner'], $arguments['repo'], $arguments['issue_number'], $params); } + /** @return */ public function addSubIssue(array $params): Issue { $arguments = []; diff --git a/clients/GitHub/src/Internal/Router/Post/Seven.php b/clients/GitHub/src/Internal/Router/Post/Seven.php index 96ed9a4e40d..8609cacc085 100644 --- a/clients/GitHub/src/Internal/Router/Post/Seven.php +++ b/clients/GitHub/src/Internal/Router/Post/Seven.php @@ -35,7 +35,7 @@ public function __construct(private Routers $routers) { } - /** @return |Observable|Schema\BasicError|Schema\Issue */ + /** @return |Observable|Schema\BasicError */ public function call(string $call, array $params, array $pathChunks): Ok|Json|WithoutBody|Created|AuthenticationToken|BranchWithProtection|EmptyObject|CodeScanningVariantAnalysis|Reaction|CommitComment|DeploymentStatus|DeploymentBranchPolicy|DeploymentProtectionRule|Issue|IssueComment|iterable|BasicError|Codespace|PullRequestReviewComment|PullRequestSimple|PullRequestReview|ReleaseAsset|\ApiClients\Client\GitHub\Schema\Operations\SecurityAdvisories\CreateRepositoryAdvisoryCveRequest\Response\ApplicationJson\Accepted\Application\Json|FullRepository { if ($pathChunks[0] === '') { diff --git a/clients/GitHub/src/Operation/Issues.php b/clients/GitHub/src/Operation/Issues.php index 232c78b5279..85481627bd7 100644 --- a/clients/GitHub/src/Operation/Issues.php +++ b/clients/GitHub/src/Operation/Issues.php @@ -236,28 +236,31 @@ public function unlock(string $owner, string $repo, int $issueNumber): WithoutBo return $this->operators->issues👷Unlock()->call($owner, $repo, $issueNumber); } + /** @return */ public function removeSubIssue(string $owner, string $repo, int $issueNumber, array $params): Issue { return $this->operators->issues👷RemoveSubIssue()->call($owner, $repo, $issueNumber, $params); } - /** @return iterable */ + /** @return Observable */ public function listSubIssues(string $owner, string $repo, int $issueNumber, int $perPage, int $page): iterable { return $this->operators->issues👷ListSubIssues()->call($owner, $repo, $issueNumber, $perPage, $page); } - /** @return iterable */ + /** @return Observable */ public function listSubIssuesListing(string $owner, string $repo, int $issueNumber, int $perPage, int $page): iterable { return $this->operators->issues👷ListSubIssuesListing()->call($owner, $repo, $issueNumber, $perPage, $page); } + /** @return */ public function addSubIssue(string $owner, string $repo, int $issueNumber, array $params): Issue { return $this->operators->issues👷AddSubIssue()->call($owner, $repo, $issueNumber, $params); } + /** @return */ public function reprioritizeSubIssue(string $owner, string $repo, int $issueNumber, array $params): Issue { return $this->operators->issues👷ReprioritizeSubIssue()->call($owner, $repo, $issueNumber, $params); diff --git a/clients/GitHub/src/PHPStan/ClientCallReturnTypes.php b/clients/GitHub/src/PHPStan/ClientCallReturnTypes.php index 9cecfc4ad3d..7c4ef1befd4 100644 --- a/clients/GitHub/src/PHPStan/ClientCallReturnTypes.php +++ b/clients/GitHub/src/PHPStan/ClientCallReturnTypes.php @@ -3136,23 +3136,23 @@ public function getTypeFromMethodCall(MethodReflection $methodReflection, Method } if ($call === 'DELETE /repos/{owner}/{repo}/issues/{issue_number}/sub_issue') { - return $this->typeResolver->resolve('Schema\\Issue'); + return $this->typeResolver->resolve(''); } if ($call === 'GET /repos/{owner}/{repo}/issues/{issue_number}/sub_issues') { - return $this->typeResolver->resolve('iterable'); + return $this->typeResolver->resolve('Observable'); } if ($call === 'LIST /repos/{owner}/{repo}/issues/{issue_number}/sub_issues') { - return $this->typeResolver->resolve('iterable'); + return $this->typeResolver->resolve('Observable'); } if ($call === 'POST /repos/{owner}/{repo}/issues/{issue_number}/sub_issues') { - return $this->typeResolver->resolve('Schema\\Issue'); + return $this->typeResolver->resolve(''); } if ($call === 'PATCH /repos/{owner}/{repo}/issues/{issue_number}/sub_issues/priority') { - return $this->typeResolver->resolve('Schema\\Issue'); + return $this->typeResolver->resolve(''); } if ($call === 'GET /repos/{owner}/{repo}/issues/{issue_number}/timeline') { diff --git a/clients/GitHub/tests/Types/ClientCallReturnTypes.php b/clients/GitHub/tests/Types/ClientCallReturnTypes.php index 77cb5f7a53e..8be1cfe6681 100644 --- a/clients/GitHub/tests/Types/ClientCallReturnTypes.php +++ b/clients/GitHub/tests/Types/ClientCallReturnTypes.php @@ -790,11 +790,11 @@ function authHeader(): string assertType('Observable', $client->call('LIST /repos/{owner}/{repo}/issues/{issue_number}/reactions')); assertType('', $client->call('POST /repos/{owner}/{repo}/issues/{issue_number}/reactions')); assertType('', $client->call('DELETE /repos/{owner}/{repo}/issues/{issue_number}/reactions/{reaction_id}')); -assertType('Schema\\Issue', $client->call('DELETE /repos/{owner}/{repo}/issues/{issue_number}/sub_issue')); -assertType('iterable', $client->call('GET /repos/{owner}/{repo}/issues/{issue_number}/sub_issues')); -assertType('iterable', $client->call('LIST /repos/{owner}/{repo}/issues/{issue_number}/sub_issues')); -assertType('Schema\\Issue', $client->call('POST /repos/{owner}/{repo}/issues/{issue_number}/sub_issues')); -assertType('Schema\\Issue', $client->call('PATCH /repos/{owner}/{repo}/issues/{issue_number}/sub_issues/priority')); +assertType('', $client->call('DELETE /repos/{owner}/{repo}/issues/{issue_number}/sub_issue')); +assertType('Observable', $client->call('GET /repos/{owner}/{repo}/issues/{issue_number}/sub_issues')); +assertType('Observable', $client->call('LIST /repos/{owner}/{repo}/issues/{issue_number}/sub_issues')); +assertType('', $client->call('POST /repos/{owner}/{repo}/issues/{issue_number}/sub_issues')); +assertType('', $client->call('PATCH /repos/{owner}/{repo}/issues/{issue_number}/sub_issues/priority')); assertType('Observable', $client->call('GET /repos/{owner}/{repo}/issues/{issue_number}/timeline')); assertType('Observable', $client->call('LIST /repos/{owner}/{repo}/issues/{issue_number}/timeline')); assertType('Observable', $client->call('GET /repos/{owner}/{repo}/keys')); diff --git a/etc/specs/GitHub/current.spec.yaml b/etc/specs/GitHub/current.spec.yaml index 16071898afe..8c1e450ab88 100644 --- a/etc/specs/GitHub/current.spec.yaml +++ b/etc/specs/GitHub/current.spec.yaml @@ -9516,7 +9516,9 @@ paths: get: summary: List organization webhooks description: |- - You must be an organization owner to use this endpoint. + List webhooks for an organization. + + The authenticated user must be an organization owner to use this endpoint. OAuth app tokens and personal access tokens (classic) need `admin:org_hook` scope. OAuth apps cannot list, view, or edit webhooks that they did not create and users cannot list, view, or edit webhooks that were created by OAuth apps. @@ -9778,7 +9780,9 @@ paths: delete: summary: Delete an organization webhook description: |- - You must be an organization owner to use this endpoint. + Delete a webhook for an organization. + + The authenticated user must be an organization owner to use this endpoint. OAuth app tokens and personal access tokens (classic) need `admin:org_hook` scope. OAuth apps cannot list, view, or edit webhooks that they did not create and users cannot list, view, or edit webhooks that were created by OAuth apps. @@ -31037,7 +31041,10 @@ paths: subcategory: webhooks delete: summary: Delete a repository webhook - description: '' + description: |- + Delete a webhook for an organization. + + The authenticated user must be a repository owner, or have admin access in the repository, to delete the webhook. tags: - repos operationId: repos/delete-webhook