◐ Shell
clean mode source ↗

PSAvoidUsingPositionalParameters: Do not warn on AZ CLI by bergmeister · Pull Request #1846 · PowerShell/PSScriptAnalyzer

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@bergmeister

@bergmeister bergmeister commented

Sep 22, 2022

edited

Loading

Copy link Copy Markdown

Collaborator

PR Summary

Fixes #1845
AZ CLI entrypoint recently changed from a batch file to an az.ps1 'script' in latest version 2.40.0, which just passed $args arguments as-is to python so it is still a CLI and not a script with parameters.

PR Checklist

@bergmeister bergmeister added this to the 1.21 milestone

Sep 22, 2022

@bergmeister

Copy link Copy Markdown

Collaborator Author

After an internal review we decided that we should not have it in code but as configuration but can still exclude az by default.

@JamesWTruher JamesWTruher 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.

this looks really good

bergmeister reacted with hooray emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@JamesWTruher JamesWTruher JamesWTruher approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Projects

None yet

Milestone

1.21

Development

Successfully merging this pull request may close these issues.

PSAvoidUsingPositionalParameters reports issue with az command when version 2.40.0 of az cli is installed

2 participants

@bergmeister @JamesWTruher