Skip to content

update error message outputs and handling#31

Merged
garretpatten merged 2 commits into
masterfrom
error-messages
May 1, 2026
Merged

update error message outputs and handling#31
garretpatten merged 2 commits into
masterfrom
error-messages

Conversation

@garretpatten
Copy link
Copy Markdown
Owner

@garretpatten garretpatten commented May 1, 2026

Pull request

Changes

  • src/action.ts
    • Added buildFailureMessage() and logUnownedFilesForJob() so failures stay consistent and each unowned path is logged under a group titled Files without an effective CODEOWNERS owner (easy to search in the Actions UI).
    • Sets unownedFiles (newline-separated paths) and errorMessage before core.setFailed() so outputs are recorded reliably for later steps.
  • action.yml
    • Documented the new unownedFiles output.
  • .github/workflows/codeowner-verifier.yaml
    • Builds the comment from unownedFiles first (markdown bullets), then errorMessage, then a short fallback that names the log group to search.
    • Wraps the whole alert in blockquoteMarkdown() so every line is a proper > … blockquote (including GitHub’s [!WARNING] syntax).

Checklist

  • Any related issues are assigned to the PR author.
  • All related issues are linked from the PR.
  • Changes have been reviewed and tested locally (npm test, and npm run build when src/ or action metadata changed).
  • If src/action.ts or dependencies changed, dist/index.js is updated and included in the PR.

@garretpatten garretpatten merged commit 240e7ee into master May 1, 2026
18 checks passed
@garretpatten garretpatten deleted the error-messages branch May 1, 2026 17:11
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.

1 participant