Skip to content

fix: support Bash 3 in the dependency checkout helper - #1020

Merged
Pigbibi merged 1 commit into
mainfrom
fix/promotion-authority-read-bash3
Nov 1, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
fix/promotion-authority-read-bash3

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

AI assistance

OpenAI Codex.

What changed

Keep the internal dependency checkout helper compatible with macOS Bash 3.2 by replacing mapfile with a while read loop. Preserve quoting and the validated repository list.

This PR targets #1010. Its original duplicate promotion-authority read correction now lives in #1010 so that the base can pass CI independently. After rebasing, this PR contains only the Bash compatibility change; #1018 builds on this branch.

How verified

  • Passed the full Python suite on the final backend stack: 1,005 tests, 10 skipped, including all 13 Bash helper tests on macOS Bash 3.2. Current remote code checks and governance checks also pass.
  • The promoted authority correction passed the evidence-metadata and decision-cockpit API tests on feat(console): add decision cockpit backend API #1010.
  • No labels, merge, deployment, IAM or secret changes.

@coderabbitai

coderabbitai Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 34ded648-5be4-4bd8-9530-8852c3ced367
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@Pigbibi
Pigbibi force-pushed the fix/promotion-authority-read-bash3 branch from 9e5817b to c2f310e Compare November 1, 2026 07:44
@Pigbibi Pigbibi changed the title fix: reuse promotion authority and support Bash 3 helpers fix: support Bash 3 in the dependency checkout helper Nov 1, 2026
Base automatically changed from feat/cockpit-backend-api to main November 1, 2026 10:42
Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi force-pushed the fix/promotion-authority-read-bash3 branch from aec644f to f0277be Compare November 1, 2026 10:45
@Pigbibi
Pigbibi merged commit c4d0289 into main Nov 1, 2026
14 checks passed
@Pigbibi
Pigbibi deleted the fix/promotion-authority-read-bash3 branch November 1, 2026 10:50
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.

1 participant