Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix /updpkgsums #83

Merged
merged 3 commits into from
Jun 8, 2024
Merged

Fix /updpkgsums #83

merged 3 commits into from
Jun 8, 2024

Conversation

dscho
Copy link
Member

@dscho dscho commented Jun 7, 2024

Even after git-for-windows/git-for-windows-automation#82, the /updpkgsums call over in git-for-windows/build-extra#558 failed.

While this PR won't fix it completely, it fixes the GitForWindowsHelper GitHub App side of it.

dscho added 2 commits June 7, 2024 10:45
The most common use case for the `get-webhook-event-payload` script is
to obtain the record of a failed slash command.

To make it easier to use this script, then, accept the URL to the
comment in the PR that contains said slash command.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho self-assigned this Jun 7, 2024
When using a bare number as input, GitHub reacts with 422 as
`statusCode` and the `statusMessage` "Unprocessable Entity".

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the fix-updpkgsums branch from 8880d1e to bd8d28a Compare June 7, 2024 22:19
@dscho dscho merged commit c4d5de9 into git-for-windows:main Jun 8, 2024
1 check passed
@dscho dscho deleted the fix-updpkgsums branch June 8, 2024 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant