Skip to content

Re-pin dependabot-sync past the bot-author fix - #196

Merged
jeremy merged 1 commit into
masterfrom
ci/repin-fdf2ae72
Jul 27, 2026
Merged

Re-pin dependabot-sync past the bot-author fix#196
jeremy merged 1 commit into
masterfrom
ci/repin-fdf2ae72

Conversation

@jeremy

@jeremy jeremy commented Jul 27, 2026

Copy link
Copy Markdown
Member

basecamp/.github#12 fixed the app/<slug> bot-author constant that basecamp-sdk's full-cycle exercise tripped (fail-closed, auto-merge refused as designed). Pin bump only; the workflow entry remains disabled in this repo until its turn in the one-at-a-time rollout.


Summary by cubic

Re-pin the shared dependabot-sync-actions-comments workflow to the version with the bot-author fix so PRs from app/github-actions are handled correctly, and set permissions.contents to write to allow auto-merge updates. The workflow remains disabled in this repo until its rollout turn.

Written for commit 5458beb. Summary will update on new commits.

Review in cubic

Copilot AI review requested due to automatic review settings July 27, 2026 21:53
@github-actions

Copy link
Copy Markdown

Sensitive Change Detection (shadow mode)

This PR modifies control-plane files:

  • .github/workflows/dependabot-sync-actions-comments.yml

Shadow mode — this check is informational only. When activated, changes to these paths will require approval from a maintainer.

@github-actions github-actions Bot added the bug Something isn't working label Jul 27, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the pinned commit SHA for the reusable dependabot-sync-actions-comments workflow to pick up the upstream fix for bot author detection (app/<slug>), keeping this repo aligned with the organization workflow while preserving the existing (currently inactive in rollout) caller configuration.

Changes:

  • Re-pin the called reusable workflow from basecamp/.github to the merge commit that includes basecamp/.github#12.
  • No behavioral changes in this repo’s workflow wiring besides the upstream workflow version bump.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.


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

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Re-trigger cubic

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

basecamp/.github#12: gh reports bot PR creators as app/github-actions;
the fail-closed re-verification refused auto-merge on the exercise's
repair PR in basecamp-sdk. Workflow remains disabled in this repo until
its rollout turn.
@jeremy
jeremy force-pushed the ci/repin-fdf2ae72 branch from 0131179 to 5458beb Compare July 27, 2026 22:09
Copilot AI review requested due to automatic review settings July 27, 2026 22:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@jeremy
jeremy merged commit 3ac1de7 into master Jul 27, 2026
21 checks passed
@jeremy
jeremy deleted the ci/repin-fdf2ae72 branch July 27, 2026 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants