Skip to content

Bugfix: Update-PSScriptFileInfo should populate all #Requires lines/properties#1863

Merged
anamnavi merged 3 commits intomasterfrom
bugfix-updatepsscriptinfo-requires
Sep 8, 2025
Merged

Bugfix: Update-PSScriptFileInfo should populate all #Requires lines/properties#1863
anamnavi merged 3 commits intomasterfrom
bugfix-updatepsscriptinfo-requires

Conversation

@anamnavi
Copy link
Member

@anamnavi anamnavi commented Aug 26, 2025

Currently Update-PSScriptFileInfo populates only the RequiredModules field for #Requires statements, but there are other types of #Requires statements based on different properties that are possible, such as IsRequiredElevation, RequiredPSVersion and more in the docs here

PR Summary

PR Context

Resolves #1832

PR Checklist

@anamnavi
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@anamnavi
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@anamnavi anamnavi merged commit 6bbdc0c into master Sep 8, 2025
12 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.

Update-PSScriptFileInfo removes #requires comments

2 participants