Skip to content

Commit

Permalink
[chore][Weekly Report] Remove duplicate information (#32934)
Browse files Browse the repository at this point in the history
**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.
  • Loading branch information
crobert-1 authored May 15, 2024
1 parent 7ef91f9 commit c7ae39b
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions .github/workflows/scripts/generate-weekly-report.js
Original file line number Diff line number Diff line change
Expand Up @@ -240,19 +240,6 @@ function generateReport({ issuesData, previousReport, componentData }) {
}
out.push('</ul>');

// add json data
out.push('\n ## JSON Data');
out.push('<!-- MACHINE GENERATED: DO NOT EDIT -->');
out.push(`<details>
<summary>Expand</summary>
<pre>
{
"issuesData": ${JSON.stringify(issuesData, null, 2)},
"componentData": ${JSON.stringify(componentData, null, 2)}
}
</pre>
</details>`);
const report = out.join('\n');
return report;
}

Expand Down

0 comments on commit c7ae39b

Please sign in to comment.