Skip to content

deal with case where the tagbot comment is too long#24647

Merged
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:length
Nov 21, 2025
Merged

deal with case where the tagbot comment is too long#24647
Micket merged 1 commit intoeasybuilders:developfrom
branfosj:length

Conversation

@branfosj
Copy link
Copy Markdown
Member

@branfosj branfosj commented Nov 21, 2025

In #24469 there is no comparison with existing easyconfigs because the comment is too long:

Failed to post comment: 422, {"message":"Validation Failed","errors":[{"resource":"IssueComment","code":"unprocessable","field":"data","message":"Body is too long (maximum is 65536 characters)"}],"documentation_url":"https://docs.github.com/rest/issues/comments#create-an-issue-comment","status":"422"}

Micket
Micket approved these changes Nov 21, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@Micket Micket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Micket Micket added this to the next release (5.2.0?) milestone Nov 21, 2025
@Micket Micket merged commit 3d4ff6e into easybuilders:develop Nov 21, 2025
8 checks passed
@Micket
Copy link
Copy Markdown
Contributor

Micket commented Nov 21, 2025

oh crap, there is a bug, it should be len(comment)

@Micket Micket mentioned this pull request Nov 21, 2025
Merged
@branfosj branfosj deleted the length branch November 21, 2025 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Micket Micket Micket approved these changes

Assignees

No one assigned

Labels

workflow

Projects

None yet

Milestone

5.2.0

Development

Successfully merging this pull request may close these issues.

2 participants

@branfosj @Micket