Fix source.dot.net branding and fetch the VMR manifest via git - #268
Merged
tannergooding merged 2 commits intoJul 19, 2026
Merged
Conversation
ApplyScriptsJsCustomizations only rewrote index/scripts.js, but the proxy deployment (Helpers.ServeProxiedIndex) serves scripts.js locally from wwwroot, so source.dot.net kept serving the un-rewritten template with branding off. Write the rewritten script to both copies. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
The stage1 bundle doesn't ship src/source-manifest.json, so fetch just that file from dotnet/dotnet at the pinned commit with a treeless single-file partial clone, keyed to the full dotnet-dotnet vertical's SHA rather than the partial dotnet-dotnet-windows one. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
tannergooding
force-pushed
the
tannergooding-investigate-azure-file-loading
branch
from
July 19, 2026 06:52
2d5e2a8 to
3e6c85e
Compare
tannergooding
changed the title
jeffhandley
approved these changes
Jul 19, 2026
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
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two source.dot.net fixes on top of #267.
Fetch the VMR source manifest from git for sub-repo tags
The per-subfolder repo filter tags for the dotnet/dotnet VMR came up empty because the stage1 bundle doesn't ship
src/source-manifest.json(it isn't a project file), and the extracted vertical can be the partialdotnet-dotnet-windowsbundle. Fetch just that file from dotnet/dotnet at the pinned commit via a treeless single-file partial clone, keyed to the fulldotnet-dotnetvertical's SHA rather than the windows one. Source links are unaffected — this only feeds/repoPathfiltering.Ensure
/showBrandingsurvives deploymentThe branding toggle was only rewritten into
index/scripts.js, but the proxy serveswwwroot/scripts.jsfor site chrome, so the deployed site kept the rawfalsetoken. The rewrite now patches both copies.