#4 Add support for output variables#12
Merged
ndivho-makhuvha merged 6 commits intoJun 8, 2026
Merged
Conversation
ndivho-makhuvha
requested review from
miroslavpojer and
oto-macenauer-absa
as code owners
There was a problem hiding this comment.
Pull request overview
Adds structured GitHub Action outputs (result, fail_count, warn_count) and a fail_on_warn input so workflows can both branch programmatically on validation metrics and optionally treat warnings as failures.
Changes:
- Document
fail_on_warninput and new outputs inREADME.md, including conditional-workflow examples. - Declare
outputs:inaction.ymland emit values via$GITHUB_OUTPUT. - Enhance Step Summary with a small metrics table (failed/warning counts).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
README.md |
Documents new input/output contract and shows how to use outputs in workflows. |
action.yml |
Adds fail_on_warn input, declares outputs, emits outputs, and prints summary metrics. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
oto-macenauer-absa
previously approved these changes
Jun 5, 2026
oto-macenauer-absa
left a comment
oto-macenauer-absa
left a comment
Collaborator
There was a problem hiding this comment.
gj, just minor comments
Co-authored-by: Oto Macenauer <112095903+oto-macenauer-absa@users.noreply.github.com>
ndivho-makhuvha
deleted the
feat/expose-structured-action-outputs-and-add-fail-on-warn-input
branch
oto-macenauer-absa
added a commit
that referenced
this pull request
Jun 8, 2026
The refactor that extracted inline bash to validate.sh dropped fail_on_warn support, output variable emissions, and the metrics table from the step summary. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview
Adds
fail_countandwarn_countoutput values for programmatic access to validation metrics.Adds
fail_on_warninput variable to optionally fail the action when certificates are nearing expiry.Release Notes
fail_countandwarn_countoutputs to enable programmatic access to validation metricsfail_on_warninput parameter to optionally fail the action when certificates are nearing expiryRelated
Closes #4