Skip to content

fix: Always return empty releaseNotes endpoint response#214

Merged
sh4sm merged 1 commit intomainfrom
fix/empty-release-page-response
Apr 29, 2026
Merged

fix: Always return empty releaseNotes endpoint response#214
sh4sm merged 1 commit intomainfrom
fix/empty-release-page-response

Conversation

@sh4sm
Copy link
Copy Markdown
Contributor

@sh4sm sh4sm commented Apr 29, 2026

What this PR does / why we need it:
Quick fix now to avoid issues with too large release pages for the Github API. Also the previous response required a manual clean up anyway, so avoid publishing these false-positives, which could be picked up by external parties like VAS.

@sh4sm sh4sm force-pushed the fix/empty-release-page-response branch from f952ecf to 445d129 Compare April 29, 2026 08:23
Quick fix now to avoid issues with too large release pages for the Github API.
Also the previous response required a manual clean up anyway, so avoid publishing these
false-positives, which could be picked up by external parties like VAS.
@sh4sm sh4sm force-pushed the fix/empty-release-page-response branch from 445d129 to 8af2a46 Compare April 29, 2026 08:43
Copy link
Copy Markdown
Contributor

@moritzkirchnersap moritzkirchnersap left a comment

Choose a reason for hiding this comment

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

Please add test coverage to reach at least ~95% diff coverage

@sh4sm sh4sm merged commit 2b318ff into main Apr 29, 2026
4 checks passed
@sh4sm sh4sm deleted the fix/empty-release-page-response branch April 29, 2026 08:52
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