-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
[Weekly report] Action failed due to message body being too long #32655
Labels
Comments
crobert-1
added
needs triage
New item requiring triage
help wanted
Extra attention is needed
labels
Apr 23, 2024
I'm not sure if there's anything we can do here other than removing some of the contents from the created issue. I assume the best option is to remove the issue titles in the issues needing triage and new issues in the description. This is where most characters are coming from. We'd lose some helpful information there but I don't think it's a deal breaker. |
crobert-1
added
good first issue
Good for newcomers
and removed
needs triage
New item requiring triage
labels
Apr 23, 2024
andrzej-stencel
pushed a commit
that referenced
this issue
May 15, 2024
**Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> The weekly report is currently broken because it's trying to make requests with too large of a body. The section removed here is data the is already included in the previous sections of the report, just in less structured format, directly in JSON. I don't think this has a lot of value, so it can be removed for the sake of getting the rest of the data. This will remove the `JSON Data` section of previous [weekly reports](#32407). **Link to tracking Issue:** <Issue number if applicable> Fixes #32655 **Testing:** <Describe what testing was performed and which tests were added.> I haven't been able to test this, but the weekly report is currently broken and is only run once a week, so I don't think this could break it worse.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component(s)
No response
Describe the issue you're reporting
Failed CI/CD link
Failure output:
The text was updated successfully, but these errors were encountered: