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

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/9782
Author: @akkaygin
Created: 4/12/2021
Status: Merged
Merged: 4/21/2021
Merged by: @DHowett

Base: mainHead: doc-fix


📝 Commits (2)

  • 9acfae9 Improved the clarity of the extra step involving
  • 65ab899 Added Get-Format function to OpenConsole.psm1 and

📊 Changes

2 files changed (+16 additions, -2 deletions)

View changed files

📝 doc/building.md (+7 -1)
📝 tools/OpenConsole.psm1 (+9 -1)

📄 Description

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

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/9782 **Author:** [@akkaygin](https://github.com/akkaygin) **Created:** 4/12/2021 **Status:** ✅ Merged **Merged:** 4/21/2021 **Merged by:** [@DHowett](https://github.com/DHowett) **Base:** `main` ← **Head:** `doc-fix` --- ### 📝 Commits (2) - [`9acfae9`](https://github.com/microsoft/terminal/commit/9acfae9b052493b34eb4717eb854cdf8d643a1ed) Improved the clarity of the extra step involving - [`65ab899`](https://github.com/microsoft/terminal/commit/65ab899e813325ce34332f3cb5df93b1bb510ed9) Added Get-Format function to OpenConsole.psm1 and ### 📊 Changes **2 files changed** (+16 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `doc/building.md` (+7 -1) 📝 `tools/OpenConsole.psm1` (+9 -1) </details> ### 📄 Description <!-- 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 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:23:57 +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#27741