Skip to content

switching to variable region#1384

Merged
Lsubatin merged 5 commits into
mainfrom
lab2
Jun 23, 2026
Merged

switching to variable region#1384
Lsubatin merged 5 commits into
mainfrom
lab2

Conversation

@Lsubatin

Copy link
Copy Markdown
Contributor

No description provided.

@gemini-code-assist gemini-code-assist 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.

Code Review

This pull request replaces the hardcoded 'us-central1' region with a dynamic '$REGION' variable in 'setup_lab.sh' to support multi-region deployments, and cleans up trailing whitespace in 'setup_alloydb.sh'. The review feedback recommends wrapping the newly introduced '$REGION' and '${PROJECT_ID}' variables in double quotes across several commands to prevent word splitting and ensure script robustness.

Comment thread codelabs/bigquery-alloydb-insights/scripts/setup_lab.sh Outdated
Comment thread codelabs/bigquery-alloydb-insights/scripts/setup_lab.sh
Comment thread codelabs/bigquery-alloydb-insights/scripts/setup_lab.sh Outdated
Comment thread codelabs/bigquery-alloydb-insights/scripts/setup_lab.sh Outdated
Comment thread codelabs/bigquery-alloydb-insights/scripts/setup_lab.sh Outdated
Lsubatin and others added 4 commits June 23, 2026 19:36
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@Lsubatin Lsubatin merged commit 1c17935 into main Jun 23, 2026
10 checks passed
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