Skip to content

Conversation

@BillyONeal
Copy link
Member

  • Fixes running the node scripts on Windows
    • Sorts filenames so that rerunning generateDetailedPages.js produces no diffs
    • Fixed command line quoting to use " rather than '
  • Improve perf by not running git log on each individual line. This change was authored by GitHub Copilot CLI / gpt-5.1-codex-max
  • Allow passing in an external vcpkg repo to generateDetailedPages.js. This change was authored by GitHub Copilot CLI / gpt-5.1-codex-max

I verified that the generated website content is the same by changing the baseline SHA to something else, running generation, seeing changes, changing it back to the original SHA, rerunning generation, and seeing no diffs other than completely added ports.

* Fixes running the node scripts on Windows
    * Sorts filenames so that rerunning generateDetailedPages.js produces no diffs
    * Fixed command line quoting to use " rather than '
* Improve perf by not running git log on each individual line. This change was authored by GitHub Copilot CLI / gpt-5.1-codex-max
* Allow passing in an external vcpkg repo to generateDetailedPages.js. This change was authored by GitHub Copilot CLI / gpt-5.1-codex-max

I verified that the generated website content is the same by changing the baseline SHA to something else, running generation, seeing changes, changing it back to the original SHA, rerunning generation, and seeing no diffs other than completely added ports.
Copy link
Contributor

@vicroms vicroms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BillyONeal BillyONeal merged commit 4588ad5 into vcpkg:main Jan 7, 2026
1 check passed
@BillyONeal BillyONeal deleted the improve-website-speed branch January 7, 2026 00:17
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