Skip to content

CI/GnuComment: add safeguards#11793

Merged
sylvestre merged 1 commit intouutils:mainfrom
xtqqczze:GH11763
Apr 13, 2026
Merged

CI/GnuComment: add safeguards#11793
sylvestre merged 1 commit intouutils:mainfrom
xtqqczze:GH11763

Conversation

@xtqqczze
Copy link
Copy Markdown
Contributor

@xtqqczze xtqqczze commented Apr 13, 2026

Fix #11763 (comment)

Summary

  • check for payload existence
  • check for PR number

script: |
const fs = require('fs');
if (!fs.existsSync('./NR') || !fs.existsSync('./result.txt')) {
core.info("No GNU comment payload to post.");
Copy link
Copy Markdown
Contributor Author

@xtqqczze xtqqczze Apr 13, 2026

Choose a reason for hiding this comment

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

@sylvestre If there is no payload should we edit the existing comment to remove content?

@xtqqczze xtqqczze changed the title CI/GnuComment: check for payload existence CI/GnuComment: add safeguards Apr 13, 2026
@sylvestre sylvestre merged commit 44aff6b into uutils:main Apr 13, 2026
167 checks passed
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