Skip to content

fix(gen): resolve lint errors in generated models#2

Merged
jagtejsodhi merged 1 commit into
nextfrom
llois-c/fix-gen-lint
May 9, 2026
Merged

fix(gen): resolve lint errors in generated models#2
jagtejsodhi merged 1 commit into
nextfrom
llois-c/fix-gen-lint

Conversation

@lucasloisp
Copy link
Copy Markdown
Collaborator

@lucasloisp lucasloisp commented May 8, 2026

Add missing Status enum to BridgeFiatCustomerResponse and
BridgeSandboxFiatCustomerResponse (referenced but undefined), and
replace the broken Transaction::Details subclass in
Wallets::TransactionGetResponse — which tried to inherit from the
TransactionDetail union module — with a direct union reference.

Do not merge: should be fixed as part of generation.

Co-Authored-By: Claude Opus 4.7 noreply@anthropic.com
Committed-By-Agent: claude

Add missing `Status` enum to `BridgeFiatCustomerResponse` and
`BridgeSandboxFiatCustomerResponse` (referenced but undefined), and
replace the broken `Transaction::Details` subclass in
`Wallets::TransactionGetResponse` — which tried to inherit from the
`TransactionDetail` union module — with a direct union reference.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Committed-By-Agent: claude
Copy link
Copy Markdown

@jagtejsodhi jagtejsodhi left a comment

Choose a reason for hiding this comment

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

Discussed offline, plan is to merge as is, but can revert custom changes once Stainless fixes generation pipeline.

@jagtejsodhi jagtejsodhi marked this pull request as ready for review May 8, 2026 21:12
Copy link
Copy Markdown

jagtejsodhi commented May 8, 2026

Merge activity

  • May 8, 9:59 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 8, 10:00 PM UTC: Graphite couldn't merge this PR because it failed for an unknown reason (Merge commits are not allowed on this repository. Please update the merge method in your Graphite merge queue settings, or enable merge commits in your GitHub repository settings.).
  • May 9, 12:48 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 9, 12:48 AM UTC: @jagtejsodhi merged this pull request with Graphite.

@jagtejsodhi jagtejsodhi merged commit 5ed45a4 into next May 9, 2026
13 checks passed
@stainless-app stainless-app Bot mentioned this pull request May 9, 2026
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