[PR #9782] doc, tools: Improve docs around using clang-format with VS #27746

Open
opened 2026-01-31 09:23:58 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/9782

State: closed
Merged: Yes


Summary of the Pull Request

Commit dbbe820ae4 seems to change where OpenConsole.psm1 puts clang-format.exe but /doc/building.md still uses the path from commit 9b92986b49.
Updated clang-format.exe's path and added the required steps to generate a clang-format.exe file.

References

PR Checklist

  • Closes https://github.com/microsoft/terminal/issues/9777
  • CLA signed. If not, go over here and sign the CLA
  • Tests added/passed
  • Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx (building.md doesn't exist in docs repo)
  • Schema updated.
  • I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx
**Original Pull Request:** https://github.com/microsoft/terminal/pull/9782 **State:** closed **Merged:** Yes --- <!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request Commit https://github.com/microsoft/terminal/commit/dbbe820ae40d208c07c45b700fcc99063e68d1eb seems to change where OpenConsole.psm1 puts clang-format.exe but /doc/building.md still uses the path from commit https://github.com/microsoft/terminal/commit/9b92986b49bed8cc41fde4d6ef080921c41e6d9e. Updated clang-format.exe's path and added the required steps to generate a clang-format.exe file. <!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> ## References <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist * [x] Closes https://github.com/microsoft/terminal/issues/9777 * [x] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA * [ ] Tests added/passed * [x] Documentation updated. If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/terminal) and link it here: #xxx (building.md doesn't exist in [docs repo](https://github.com/MicrosoftDocs/terminal)) * [ ] Schema updated. * [ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx
claunia added the pull-request label 2026-01-31 09:23:58 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#27746