chore(seed): add api-wide-base-path-with-default to allowedFailures for csharp-sdk#16014
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.
Tip: disable this comment in your organization's Code Review settings.
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
…or php-sdk and csharp-sdk Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
d1e4254 to
923fdff
Compare
SDK Generation Benchmark ResultsComparing PR branch against median of 5 nightly run(s) on Full benchmark table (click to expand)
main (generator): generator-only time via --skip-scripts (includes Docker image build, container startup, IR parsing, and code generation — this is the same Docker-based flow customers use via |
Description
The new
api-wide-base-path-with-defaultfixture introduced in #15998 causes seed test failures forcsharp-sdk. This adds the fixture toallowedFailuresuntil the generator supports the new feature.Note:
php-sdkwas already fixed on main via #16006.Changes Made
api-wide-base-path-with-defaulttoallowedFailuresinseed/csharp-sdk/seed.ymlTesting
❌ THERE WERE UNEXPECTED TEST CASE FAILURES: ... api-wide-base-path-with-defaultLink to Devin session: https://app.devin.ai/sessions/eae9060efee04cb49d8451238ffd1ce9
Requested by: @Swimburger