Skip to content

Update formatting scripts for Preview-only VS and pwsh 7#20312

Merged
DHowett merged 1 commit into
mainfrom
dev/migrie/formatting
Jun 11, 2026
Merged

Update formatting scripts for Preview-only VS and pwsh 7#20312
DHowett merged 1 commit into
mainfrom
dev/migrie/formatting

Conversation

@zadjii-msft

Copy link
Copy Markdown
Member

This is a pair of changes to make the code formatting scripts work, circa 2026.

  • If you only have VS Insiders installed, then vswhere can't actually find a clang-format to build with. You need -prerelease to let it also find preview builds of VS.
  • Years ago we updated OpenConsole.psm1 to only support pwsh 7. I, of course, use cmd as my dev env for terminal, so I use runformat.cmd, which always shelled to powershell (5). This updates that tool to use pwsh 7. Clearly this impacted only me ever.

related to #20162

This is a pair of changes to make the code formatting scripts work, circa 2026.

* If you only have VS Insiders installed, then vswhere can't actually find a clang-format to build with. You need `-prerelease` to let it also find preview builds of VS.
* Years ago we updated OpenConsole.psm1 to only support pwsh 7. I, of course, use `cmd` as my dev env for terminal, so I use `runformat.cmd`, which always shelled to `powershell` (5). This updates that tool to use pwsh 7. Clearly this impacted only me ever.

related to #20162
@zadjii-msft

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@DHowett DHowett merged commit 9853bc9 into main Jun 11, 2026
12 checks passed
@DHowett DHowett deleted the dev/migrie/formatting branch June 11, 2026 22:02
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