Skip to content

ci(trunk): refresh caller workflow#43

Open
ss-o wants to merge 2 commits into
mainfrom
issue-408-trunk-caller
Open

ci(trunk): refresh caller workflow#43
ss-o wants to merge 2 commits into
mainfrom
issue-408-trunk-caller

Conversation

@ss-o
Copy link
Copy Markdown
Member

@ss-o ss-o commented May 19, 2026

Summary

  • Update the Trunk caller workflow to the refreshed org-level single-job shape from ci(trunk): refresh reusable workflow .github#404.
  • Remove stale commented upload-token wiring from the caller.
  • Add explicit permissions, concurrency, and arguments: --no-progress.

Part of z-shell/.github#408.

Verification

  • Workflow-only change to .github/workflows/trunk-check.yml.
  • YAML shape reviewed after writing the workflow.

Agent handoff

Status: Ready for review
Current state: PR contains only the Trunk caller refresh.
Blockers: None known.
Next steps: Wait for PR CI; merge if green.

Copilot AI review requested due to automatic review settings May 19, 2026 04:19
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refreshes this repo’s Trunk “caller” workflow to match the updated org-level reusable-workflow invocation pattern, with explicit permissions/concurrency and a simplified single job.

Changes:

  • Renames the workflow/job and expands pull_request triggers to include reopened.
  • Adds explicit workflow/job permissions plus concurrency cancellation.
  • Removes the stale commented “upload-token” wiring and passes arguments: --no-progress to the reusable workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 21 to +27
jobs:
check:
if: github.event.schedule != '0 07 * * 5'
name: "⚡"
name: Trunk Code Quality
permissions:
actions: read
checks: write
contents: read
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.

2 participants