Probe: Stainless preview vs csharp-free base (throwaway)#659
Probe: Stainless preview vs csharp-free base (throwaway)#659ls-bolt[bot] wants to merge 1 commit into
Conversation
…view? Doc-comment-only delta (proven to fail preview when based on main). This PR's merge base lacks the csharp target, so if the Stainless preview check goes green here, the compare API unions targets across base and head — meaning the csharp removal fixes the check once it reaches main. Throwaway probe; will be closed either way. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. 2 Skipped Deployments
|
✱ Stainless preview builds for gridThis PR will update the cli csharp go kotlin openapi php python ruby typescript Edit this comment to update them. They will appear in their respective SDK's changelogs. ✅ grid-openapi studio · code · diff
✅ grid-ruby studio · code · diff
✅ grid-go studio · code · diff
✅ grid-kotlin studio · code · diff
✅ grid-typescript studio · code · diff
✅ grid-python studio · code · diff
✅ grid-csharp studio · code · diff
✅ grid-php studio · code · diff
✅ grid-cli studio · code · diff
This comment is auto-generated by GitHub Actions and is automatically kept up to date as you push. |
|
Probe complete: preview PASSED (see #657 for the full verdict). csharp still built and its build CI still failed, but base failed identically → 'did not represent a regression' → green. Closing. |
Throwaway probe for the csharp
previewinvestigation on #657 — do not review or merge.Question: does
builds.compareunion the target set across base and head? On #657, the head config verifiably lacked the csharp target yet the server still built csharp — the suspected reason is that the base build (from the merge base = main, which still has csharp) forces csharp into the comparison.Setup: this PR is based on
07-06-pan-embed-url-fetch-on-demand-docs(csharp-free config on both sides) and carries a doc-comment-only codegen delta (GtqAccountInfoBase.bankAccountTypedescription) — the exact class of change proven to failpreviewagainst a main base.Read-out:
previewgreen → union confirmed; the csharp removal in docs(cards): panEmbedUrl is nullable and minted only on GET /cards/{id} #657 fixes the check for all PRs once it reaches main.previewred on csharp → the target set persists server-side in Stainless Studio regardless of config; repo-side fixes are impossible and someone with Studio access must remove the target.Will report the result on #657 and close this PR.
🤖 thundering-graviton(#2) | Feedback