Skip to content

chore: add shared ltx workflow callers [DOPS-392] - #1

Merged
kevnm67 merged 2 commits into
mainfrom
chore/DOPS-392-add-shared-workflows
Apr 9, 2026
Merged

chore: add shared ltx workflow callers [DOPS-392]#1
kevnm67 merged 2 commits into
mainfrom
chore/DOPS-392-add-shared-workflows

Conversation

@kevnm67

@kevnm67 kevnm67 commented Apr 9, 2026

Copy link
Copy Markdown
Member

Summary

  • Add shared merge-on-green and claude workflow callers from LocumTenens-com/ltx
  • Enables LGTM-to-merge and @claude mention support

Test plan

  • LGTM comment triggers merge via reusable workflow
  • @claude mention triggers Claude interactive session

🤖 Generated with Claude Code

@qltysh

qltysh Bot commented Apr 9, 2026

Copy link
Copy Markdown

❌ 5 blocking issues (5 total)

Tool Category Rule Count
zizmor Vulnerability overly broad permissions 4 ❌
actionlint Lint undefined function "trim". available functions are "always", "cancelled", "contains", "endswith", "failure", "format", "fromjson", "hashfiles", "join", "startswith", "success", "tojson" 1 ❌

qltysh[bot]
qltysh Bot reviewed Apr 9, 2026
View reviewed changes
Comment thread .github/workflows/claude.yml
with:
pr-number: ${{ github.event.issue.number || github.event.pull_request.number || '' }}
secrets:
ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}

@qltysh qltysh Bot Apr 9, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

overly broad permissions [zizmor:zizmor/excessive-permissions]

Comment thread .github/workflows/merge-on-green.yml
event-type: labeled
commenter: ${{ github.event.sender.login }}
secrets:
ANTHROPIC_API_KEY: ${{ secrets.ANTHROPIC_API_KEY }}

@qltysh qltysh Bot Apr 9, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Found 2 issues:

1. overly broad permissions [zizmor:zizmor/excessive-permissions]


2. overly broad permissions [zizmor:zizmor/excessive-permissions]

Comment thread .github/workflows/merge-on-green.yml
pr-number: ${{ github.event.issue.number || github.event.pull_request.number }}
event-type: ${{ github.event_name == 'issue_comment' && 'comment' || 'review' }}
commenter: ${{ github.event.comment.user.login || github.event.review.user.login }}
secrets:

@qltysh qltysh Bot Apr 9, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

overly broad permissions [zizmor:zizmor/excessive-permissions]

Comment thread .github/workflows/merge-on-green.yml
if: |
(
github.event_name == 'issue_comment' &&
github.event.issue.pull_request &&

@qltysh qltysh Bot Apr 9, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

undefined function "trim". available functions are "always", "cancelled", "contains", "endswith", "failure", "format", "fromjson", "hashfiles", "join", "startswith", "success", "tojson" [actionlint:expression]

@kevnm67
kevnm67 merged commit e0edc2e into main Apr 9, 2026
2 checks passed
@kevnm67
kevnm67 deleted the chore/DOPS-392-add-shared-workflows branch April 9, 2026 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qltysh qltysh[bot] qltysh[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kevnm67