Skip to content

Update path for PowerShell Script Analyzer - #2204

Open
Travis Plunk (TravisEz13) wants to merge 2 commits into
PowerShell:mainfrom
TravisEz13:patch-2
Open

Travis Plunk (TravisEz13) wants to merge 2 commits into
PowerShell:mainfrom
TravisEz13:patch-2

Conversation

@TravisEz13

Copy link
Copy Markdown
Member

PR Summary

This pull request updates the routing configuration for the es-metadata.yml file, specifically refining the default area path under the providers section to point to a more specific directory.

Configuration update:

  • Changed the defaultAreaPath under providers to route to One\MGMT\Compute\Powershell\Powershell\PowerShell Core\Script Analyzer instead of the broader One\MGMT\Compute\Powershell\Powershell path.

PR Checklist

Copilot AI 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.

Pull request overview

This pull request updates the es-metadata.yml routing configuration so the providers -> routing -> defaultAreaPath points to a more specific area path for PowerShell Script Analyzer.

Changes:

  • Refined defaultAreaPath.path from One\MGMT\Compute\Powershell\Powershell to One\MGMT\Compute\Powershell\Powershell\PowerShell Core\Script Analyzer.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

happy with it if you are and have testing but just to prevent any whitespace quoting issues, I suggest to get rid of whitespace and maybe consistency wise use same name as repo

Comment thread es-metadata.yml
defaultAreaPath:
org: msazure
path: One\MGMT\Compute\Powershell\Powershell
path: One\MGMT\Compute\Powershell\Powershell\PowerShell Core\Script Analyzer

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggest to use same name as repo or at least to not have whitespace to prevent potential failure where path is not quoted correctly

Suggested change
path: One\MGMT\Compute\Powershell\Powershell\PowerShell Core\Script Analyzer
path: One\MGMT\Compute\Powershell\Powershell\PowerShell Core\PSScriptAnalyzer

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