Skip to content

Fix the CI pipeline.#303

Merged
iclanton merged 3 commits intomicrosoft:mainfrom
iclanton:fix-ci-pipeline
Feb 24, 2026
Merged

Fix the CI pipeline.#303
iclanton merged 3 commits intomicrosoft:mainfrom
iclanton:fix-ci-pipeline

Conversation

@iclanton
Copy link
Member

No description provided.

The constant at module scope accessed window.devicePixelRatio, which
throws ReferenceError during Docusaurus server-side rendering in Node.js.
Added typeof window check so it safely defaults to 100 during SSR.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Move window.devicePixelRatio access inside a BrowserOnly component
wrapper as recommended. During SSR the fallback renders the avatar at
size=100; in the browser the correct size is determined from the device
pixel ratio. Also adds size as a parameter to getGitHubProfileInfo.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@iclanton iclanton enabled auto-merge February 24, 2026 06:46
@iclanton iclanton disabled auto-merge February 24, 2026 06:46
@iclanton iclanton enabled auto-merge (squash) February 24, 2026 06:46
@iclanton iclanton merged commit d1214d3 into microsoft:main Feb 24, 2026
2 checks passed
@iclanton iclanton deleted the fix-ci-pipeline branch February 24, 2026 07:36
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