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

Ensure status descriptions are not over 140 characters #239

Merged
merged 2 commits into from
Dec 11, 2024
Merged

Conversation

pbrisbin
Copy link
Member

@pbrisbin pbrisbin commented Dec 11, 2024

It may not be clear that the HALT file contents are meant to be
formatted like a commit or PR, with a reasonable-length title and
optional body below.

When folks write things as one long line, that is interpreted as title
and used as the status description, and we get:

Setting halted status for PR #37776
Error: HttpError: Validation Failed: {"resource":"Status","code":"custom","field":"description","message":"description is too long (maximum is 140 characters)"}

We can just truncate to avoid it.

It may not be clear that the `HALT` file contents are meant to be
formatted like a commit or PR, with a reasonable-length title and
optional body below.

When folks write things as one long line, that is interpreted as title
and used as the status description, and we get:

    Setting halted status for PR #37776
    Error: HttpError: Validation Failed: {"resource":"Status","code":"custom","field":"description","message":"description is too long (maximum is 140 characters)"}

We can just truncate to avoid it.
@pbrisbin pbrisbin changed the title pb/truncate Ensure status descriptions are not over 140 characters Dec 11, 2024
@pbrisbin pbrisbin requested a review from StevenXL December 11, 2024 12:51
@pbrisbin pbrisbin marked this pull request as ready for review December 11, 2024 12:51
@pbrisbin pbrisbin requested a review from a team as a code owner December 11, 2024 12:51
@pbrisbin pbrisbin requested review from derelbenkoenig and removed request for a team December 11, 2024 12:51
src/main.ts Show resolved Hide resolved
@pbrisbin pbrisbin removed the request for review from derelbenkoenig December 11, 2024 16:12
@pbrisbin pbrisbin merged commit b050128 into main Dec 11, 2024
6 checks passed
@pbrisbin pbrisbin deleted the pb/truncate branch December 11, 2024 18:34
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.

2 participants