Skip to content

Conversation

@liamk13
Copy link
Contributor

@liamk13 liamk13 commented Aug 12, 2025

Description

Updated [accessSas] to "accessSAS" as it was causing an error when running the script. It is resolved with this update

Checklist

  • Scripts in this pull request are written for the bash shell.
  • This pull request was tested on at least one of the following platforms:
    • Linux
    • Azure Cloud Shell
    • macOS
    • Windows Subsystem for Linux
  • The most recent test date and test method are recorded in the script file.
  • Scripts do not contain passwords or other secret tokens that are not randomized.
  • No deprecated commands or arguments are used. (Release notes)
  • All Azure resource identifiers which must be universally unique are guaranteed to be so.
  • Resource names use a random function to ensure scripts can be run multiple times in quick succession without error.
  • All scripts can be run in their entirely without user input.

Testing information

CLI version: 2.76.0

az --version

Extensions required:

Updated [accessSas] to "accessSAS" to avoid failures
Updated [accessSas] to "accessSAS" to avoid failures
@liamk13
Copy link
Contributor Author

liamk13 commented Aug 12, 2025

@microsoft-github-policy-service agree

@liamk13
Copy link
Contributor Author

liamk13 commented Aug 12, 2025

Hi @dbradish-microsoft could this be merged?

Copy link
Collaborator

@mikefrobbins mikefrobbins left a comment

Choose a reason for hiding this comment

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

LGTM

@mikefrobbins mikefrobbins merged commit 8542e1a into Azure-Samples:master Oct 23, 2025
1 check 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.

2 participants