Skip to content

chore: make GENERATED_DIAGRAMS_REPO configurable via env var#23380

Open
ebembi-crdb wants to merge 1 commit into
mainfrom
stanley/validate-branch-env-var
Open

chore: make GENERATED_DIAGRAMS_REPO configurable via env var#23380
ebembi-crdb wants to merge 1 commit into
mainfrom
stanley/validate-branch-env-var

Conversation

@ebembi-crdb
Copy link
Copy Markdown
Contributor

Summary

  • Makes the GENERATED_DIAGRAMS_REPO constant in validate_branch_existence.py configurable via an environment variable
  • Default remains cockroachdb/generated-diagrams (that repo is not moving orgs)
  • Provides flexibility if the repo location ever changes in the future

Part of Project Stanley (EDUENG-720, Ticket 5).

Test plan

  • Verify validate_branch_existence.py still resolves branches correctly with the default value
  • Test overriding GENERATED_DIAGRAMS_REPO env var to confirm it's picked up

🤖 Generated with Claude Code

Allows the generated-diagrams repo reference in
validate_branch_existence.py to be overridden via the
GENERATED_DIAGRAMS_REPO environment variable. Defaults to
`cockroachdb/generated-diagrams` (unchanged behavior since that
repo is not moving).

Part of Project Stanley (EDUENG-720, Ticket 5).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ebembi-crdb ebembi-crdb requested a review from a team as a code owner May 21, 2026 15:27
@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 353bc7e
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-interactivetutorials-docs/deploys/6a0f2458994a930008b0f315

@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 353bc7e
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-api-docs/deploys/6a0f245887aec40008bcf74f

@github-actions
Copy link
Copy Markdown

Files changed:

  • .github/scripts/validate_branch_existence.py

@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Netlify Preview

Name Link
🔨 Latest commit 353bc7e
🔍 Latest deploy log https://app.netlify.com/projects/cockroachdb-docs/deploys/6a0f245824488a0008c0593a
😎 Deploy Preview https://deploy-preview-23380--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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