Skip to content

Conversation

@Roopan-Microsoft
Copy link
Contributor

@Roopan-Microsoft Roopan-Microsoft commented Aug 21, 2025

Purpose

  • This pull request makes a small but important change to how the deployment script is referenced in the infrastructure code. The script URI is now dynamically constructed using the baseUrl variable, instead of being hardcoded.
  • Changed the primaryScriptUri in deploy_create_table_script.bicep to use the ${baseUrl} variable, making the script location configurable and more flexible.

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft Roopan-Microsoft changed the title Update deploy_create_table_script.bicep fix: Update deploy_create_table_script.bicep Aug 21, 2025
@Roopan-Microsoft Roopan-Microsoft added this pull request to the merge queue Aug 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 21, 2025
@Roopan-Microsoft Roopan-Microsoft changed the title fix: Update deploy_create_table_script.bicep fix: update deploy_create_table_script.bicep Aug 21, 2025
@Roopan-Microsoft Roopan-Microsoft added this pull request to the merge queue Aug 21, 2025
Merged via the queue into main with commit d6c3dad Aug 21, 2025
11 of 15 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.

3 participants