Skip to content

Compliance: missing-pr-review-mention.yml #113

Closed

Description

@don-petry

Compliance Finding

Category: ci-workflows
Severity: error
Check: missing-pr-review-mention.yml

Detail

Required workflow pr-review-mention.yml is missing

Standard Reference

standards/ci-standards.md#required-workflows

Remediation

Please review the linked standard and bring this repository into compliance.

See the full standards documentation for implementation guidance.


This issue was automatically created by the weekly compliance audit.

Activity

  1. donpetry-bot commented on May 11, 2026

    @donpetry-bot
    Contributor

    Claude finished @don-petry's task in 1m 53s —— View job • claude/issue-113-20260511-1236 • [Create PR ➔](https://github.com/petry-projects/.github-private/compare/main...claude/issue-113-20260511-1236?quick_pull=1&title=Issue%20%23113%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%23113%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)


    Fix Implemented for Issue #113

    • Gather context (CLAUDE.md, standards, existing PRs)
    • Understand required pr-review-mention.yml workflow (Tier 1 stub — copy template verbatim)
    • Implement the workflow file (.github/workflows/pr-review-mention.yml)
    • Create PR — #122
    • Self-review — file is a verbatim copy of the org template, no issues found
    • CI running (CodeQL, SonarCloud, AgentShield in progress — no failures); tagged @petry-projects/org-leads to review and merge

    What was done: Added .github/workflows/pr-review-mention.yml — a Tier 1 thin-caller stub copied verbatim from petry-projects/.github/standards/workflows/pr-review-mention.yml. The stub delegates all logic to pr-review-mention-reusable.yml@v1 in the central org repo. This resolves the compliance finding that pr-review-mention.yml was missing.
    | View PR #122

  2. don-petry commented on May 11, 2026

    @don-petry
    CollaboratorAuthor

    Claude finished @don-petry's task in 1m 32s —— View job • claude/issue-113-20260511-2018 • Create PR ➔


    Fix implemented for Issue #113

    • Gather context (CLAUDE.md, standards, existing work)
    • Understand required pr-review-mention.yml workflow — template found in petry-projects/.github/standards/workflows/pr-review-mention.yml
    • Implement the workflow file — verbatim copy of org standard template
    • Commit and push changes
    • Create PR — #126
    • Self-review — file is verbatim copy of standard; correct triggers, minimal permissions, proper uses: reference. No issues found.
    • Tagged @petry-projects/org-leads to review and merge

    What was done: Added .github/workflows/pr-review-mention.yml — a Tier 1 thin-caller stub copied verbatim from petry-projects/.github/standards/workflows/pr-review-mention.yml. The stub delegates all logic to pr-review-mention-reusable.yml@v1 in the central org repo. This resolves the compliance finding that pr-review-mention.yml was missing.

    • claude/issue-113-20260511-2018

  3. don-petry commented on May 12, 2026

    @don-petry
    CollaboratorAuthor

    Weekly Compliance Audit (2026-05-12)

    This finding is still open.

    Detail: Required workflow pr-review-mention.yml is missing

    Standard: standards/ci-standards.md#required-workflows

  4. don-petry commented on May 12, 2026

    @don-petry
    CollaboratorAuthor

    Weekly Compliance Audit (2026-05-12)

    This finding is still open.

    Detail: Required workflow pr-review-mention.yml is missing

    Standard: standards/ci-standards.md#required-workflows

  5. don-petry commented on May 12, 2026

    @don-petry
    CollaboratorAuthor

    Weekly Compliance Audit (2026-05-12)

    This finding is still open.

    Detail: Required workflow pr-review-mention.yml is missing

    Standard: standards/ci-standards.md#required-workflows

  6. don-petry commented on May 12, 2026

    @don-petry
    CollaboratorAuthor

    Weekly Compliance Audit (2026-05-12)

    This finding is still open.

    Detail: Required workflow pr-review-mention.yml is missing

    Standard: standards/ci-standards.md#required-workflows

  7. don-petry commented on May 12, 2026

    @don-petry
    CollaboratorAuthor

    Weekly Compliance Audit (2026-05-12)

    This finding is still open.

    Detail: Required workflow pr-review-mention.yml is missing

    Standard: standards/ci-standards.md#required-workflows

  8. don-petry commented on May 13, 2026

    @don-petry
    CollaboratorAuthor

    Weekly Compliance Audit (2026-05-13)

    This finding is still open.

    Detail: Required workflow pr-review-mention.yml is missing

    Standard: standards/ci-standards.md#required-workflows

  9. added a commit that references this issue on May 15, 2026
    5d87115
  10. don-petry commented on May 15, 2026

    @don-petry
    CollaboratorAuthor

    Weekly Compliance Audit (2026-05-15)

    This finding is still open.

    Detail: Required workflow pr-review-mention.yml is missing

    Standard: standards/ci-standards.md#required-workflows

  11. added 2 commits that reference this issue on May 15, 2026
    c91e6ea
    5bd169b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    compliance-auditAutomated compliance audit finding

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions