Skip to content

ci: bind check-label composite inputs via env before shell - #11025

Open
SashaMIT wants to merge 1 commit into
lightningnetwork:masterfrom
SashaMIT:ci/bind-check-label-inputs-env
Open

ci: bind check-label composite inputs via env before shell#11025
SashaMIT wants to merge 1 commit into
lightningnetwork:masterfrom
SashaMIT:ci/bind-check-label-inputs-env

Conversation

@SashaMIT

@SashaMIT SashaMIT commented Aug 5, 2026

Copy link
Copy Markdown

Summary

  • Bind inputs.skip-message and the computed label-contains result through env: before shell echo/notice steps in the check-label composite action.
  • Same class as GitHub’s documented script-injection guidance for interpolating composite action inputs into run: scripts.

Test plan

  • PRs with the configured label still set skip=true and print the skip message
  • PRs without the label still set skip=false

Made with Cursor

Pass skip-message (and label-contains result) through env: before
echo to notice/summary (script-injection class for composite inputs).

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions github-actions Bot added the severity-low Best-effort review label Aug 5, 2026
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown

🟢 PR Severity: LOW

github-actions | 1 file | 15 lines changed

🟢 Low (1 file)
  • .github/actions/check-label/action.yml - CI/CD configuration change under .github/*

Analysis

This PR only modifies a GitHub Actions composite action (.github/actions/check-label/action.yml), a small (9 additions / 6 deletions) CI/CD configuration change. It does not touch any Go source, wallet/channel/HTLC logic, or database migrations, so it falls under the LOW severity tier reserved for docs/scripts/CI configuration and warrants only a best-effort review.


To override, add a severity-override-{critical,high,medium,low} label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

severity-low Best-effort review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant