FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Refactor New-CommandDataFile script by sdwheeler · Pull Request #2197 · PowerShell/PSScriptAnalyzer · GitHub

Refactor New-CommandDataFile script - #2197

Open
Sean Wheeler (sdwheeler) wants to merge 2 commits into
PowerShell:mainfrom
sdwheeler:sdw-w600137-refactor-script
Open

Refactor New-CommandDataFile script#2197
Sean Wheeler (sdwheeler) wants to merge 2 commits into
PowerShell:mainfrom
sdwheeler:sdw-w600137-refactor-script

Conversation

Sean Wheeler (sdwheeler) commented Jul 25, 2026
edited
Loading

Copy link
Copy Markdown
Collaborator

Refactor New-CommandDataFile script

  • Fixes AB#600137

This PR rewrites the script to create a function and fixes several issues:

  • Adds comment-based help
  • Adds parameter to specify an output path for the JSON file
  • Fixes the logic to avoid duplicate data output
  • Outputs a FileInfo object for the newly created file

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Pull request overview

Refactors Utils/New-CommandDataFile.ps1 into an advanced script/function that generates the command/alias JSON data file used for compatibility-related settings, adding an OutputPath parameter and modernizing the implementation.

Changes:

  • Converts the script logic into an advanced function (New-CommandDataFile) and calls it from the script entrypoint.
  • Adds OutputPath support and uses module-qualified cmdlet invocations throughout.
  • Reworks module/command/alias enumeration and JSON emission.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Utils/New-CommandDataFile.ps1 Outdated
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL