Skip to content

chore: Upgrade to pnpm 11. - #196

Merged
cpojer merged 1 commit into
mainfrom
pnpm-11
May 8, 2026
Merged

cpojer merged 1 commit into
mainfrom
pnpm-11

Conversation

@cpojer

@cpojer cpojer commented May 8, 2026

Copy link
Copy Markdown
Member

No description provided.

@netlify

netlify Bot commented May 8, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for staging-void-cloud ready!

Name Link
🔨 Latest commit c5cac85
🔍 Latest deploy log https://app.netlify.com/projects/staging-void-cloud/deploys/69fda6205a53af0008957c22
😎 Deploy Preview https://deploy-preview-196--staging-void-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@cpojer
cpojer merged commit aba44a0 into main May 8, 2026
9 checks passed
@cpojer
cpojer deleted the pnpm-11 branch May 8, 2026 09:03
cpojer pushed a commit that referenced this pull request Sep 11, 2026
…rod domain) (#196)

The "Void Build" check-run's details_url was hardcoded
`https://${projectSlug}.void.app` at every call site, so on staging the check
run linked to the production traffic domain instead of `*.staging.void.app`
(observed live: a staging build's check-run pointed at `moonglade.void.app`).

Add a `projectUrl(config)` helper that resolves the base domain from the
per-environment `SITE_DOMAIN` binding (`env.SITE_DOMAIN ?? SITE_DOMAIN`) — the
same source the deploy-URL output uses — and route all 8 check-run details_url
emitters through it: the create POST, the 5 direct terminal finalizes, the
`finalizeCheckRunToStatus` status→conclusion helper, and cancelBuild's finalize
(which previously passed no details_url at all). Prod (SITE_DOMAIN unset or
`void.app`) is byte-identical to before.

Tests: staging create+finalize link to `{slug}.staging.void.app`; the
finalizeCheckRunToStatus PATCH path (pre-start prior-run finalize) links to
staging; unset SITE_DOMAIN falls back to prod; cancelBuild now emits details_url.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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.

1 participant