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

Prevent a big report to fail creating the issue #258

Closed
rolgalan opened this issue Apr 29, 2024 · 2 comments · Fixed by #265
Closed

Prevent a big report to fail creating the issue #258

rolgalan opened this issue Apr 29, 2024 · 2 comments · Fixed by #265
Assignees
Labels
enhancement New feature or request

Comments

@rolgalan
Copy link

Is your feature request related to a problem?

Issue creation API has a limit of 65536 characters. Whenever the report is longer, the action fails with the following error:

Error: Validation Failed: {"resource":"Issue","code":"custom","field":"body","message":"body is too long (maximum is 65536 characters)"}

This might happen easily when the amount of analyzed PRs is over ~300.

Describe the solution you'd like

I would like the action to validate the size of the report before submission, splitting the extra details into multiple comments if necessary (or just create the issue with whatever it fits) without failure.

Describe alternatives you've considered

Alternatively, the action could support a config flag to include in the report only the first two tables with the basic stats, so all the details of each single PR are not reported.

Additional context

Thanks!

@zkoppert
Copy link
Member

zkoppert commented May 1, 2024

Thanks for opening this issue! I've just been putting up with this and it hadn't crossed my mind that we could make this better. 😓 Great suggestion, I've opened up a pull request to address this.

#265

@rolgalan
Copy link
Author

rolgalan commented May 2, 2024

That was so fast, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants