Skip to content

ci: stop re-checking the published files here - #1895

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/kozlek/drop-public-checks/stop-re-checking-published-files-here--1bf5814b
Oct 7, 2026
Merged

mergify[bot] merged 1 commit into
mainfrom
devs/kozlek/drop-public-checks/stop-re-checking-published-files-here--1bf5814b

Conversation

@kozlek

@kozlek kozlek commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

The skills, install.sh and the other published files only change here
through ci-bot's sync pull request, copied as of a release from the CLI's
source, where the same checks already ran against the binary that release
ships: the skill references, shellcheck, install.sh end to end and the
spell check. Running them again here checked the same files against the
same binary, on every pull request.

CI keeps only synced-files, which stops a pull request other than the
sync from editing the published files, behind ci-gate.
scripts/check-skills.py goes with the skills job (the CLI's source
checks the skills with its own test), and _typos.toml with typos: the
files left that this repository owns are its CI and merge configuration,
AGENTS.md and LICENSE.

Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com

🤖 Generated with Claude Code

Copilot AI balanced review requested due to automatic review settings October 7, 2026 10:31
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections October 7, 2026 10:31 Failure
@mergify

mergify Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 6 merge protections satisfied — ready to merge.

Show 6 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success=ci-gate

🟢 👀 Review Requirements

  • any of:
    • #approved-reviews-by>=1
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 🔎 Reviews

  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify
mergify Bot requested a review from a team October 7, 2026 10:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

The skills, install.sh and the other published files only change here
through ci-bot's sync pull request, copied as of a release from the CLI's
source, where the same checks already ran against the binary that release
ships: the skill references, shellcheck, install.sh end to end and the
spell check. Running them again here checked the same files against the
same binary, on every pull request.

CI keeps only `synced-files`, which stops a pull request other than the
sync from editing the published files, behind `ci-gate`.
scripts/check-skills.py goes with the `skills` job (the CLI's source
checks the skills with its own test), and _typos.toml with `typos`: the
files left that this repository owns are its CI and merge configuration,
AGENTS.md and LICENSE.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Change-Id: I1bf5814b100a69d6ef0696fe174b8362d845629d
@kozlek
kozlek force-pushed the devs/kozlek/drop-public-checks/stop-re-checking-published-files-here--1bf5814b branch from d06a830 to 66f4207 Compare October 7, 2026 12:39
@kozlek

kozlek commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Revision history

# Type Changes Reason Date
1 initial d06a830 2026-10-07 12:39 UTC
2 content d06a830 → 66f4207 2026-10-07 12:39 UTC

@mergify
mergify Bot deployed to Mergify Merge Protections October 7, 2026 12:39 Active
@kozlek
kozlek marked this pull request as ready for review October 7, 2026 12:49
@mergify

mergify Bot commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-10-07 14:03 UTC · Rule: default · triggered by merge protections
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged — 2026-10-07 14:04 UTC · at b77dc4b12609a3c5b524ca9c26d128dfd42bed35 · squash

This pull request spent 14 seconds in the queue, including 2 seconds running CI.

Required conditions to merge

@mergify
mergify Bot merged commit b77dc4b into main Oct 7, 2026
4 checks passed
@mergify mergify Bot added the queued label Oct 7, 2026
@mergify
mergify Bot deleted the devs/kozlek/drop-public-checks/stop-re-checking-published-files-here--1bf5814b branch October 7, 2026 14:04
@mergify mergify Bot removed the queued label Oct 7, 2026

This branch was successfully deployed

1 active deployment
Mergify Merge Protections — 66f42070 Deployed Oct 7, 2026 by mergify[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants