Skip to content

fix(v2): openapi inject missing schemas into gnostic generator#5825

Merged
kodiakhq[bot] merged 1 commit into
v2from
rd/v2/openapi-followup
May 9, 2026
Merged

fix(v2): openapi inject missing schemas into gnostic generator#5825
kodiakhq[bot] merged 1 commit into
v2from
rd/v2/openapi-followup

Conversation

@erka

@erka erka commented May 8, 2026

Copy link
Copy Markdown
Contributor

A better workaround for #5823 without dirty things in Flipt GRPC.

@erka
erka requested a review from a team as a code owner May 8, 2026 09:50
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label May 8, 2026
@github-actions

github-actions Bot commented May 8, 2026

Copy link
Copy Markdown
Contributor

The latest Buf updates on your PR. Results from workflow Proto / proto-lint (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed❌ failed (5)May 9, 2026, 6:03 PM

@codecov

codecov Bot commented May 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.19%. Comparing base (3790369) to head (767bd46).
⚠️ Report is 2 commits behind head on v2.

Additional details and impacted files
@@           Coverage Diff           @@
##               v2    #5825   +/-   ##
=======================================
  Coverage   61.19%   61.19%           
=======================================
  Files         141      141           
  Lines       14195    14195           
=======================================
  Hits         8686     8686           
  Misses       4777     4777           
  Partials      732      732           
Flag Coverage Δ
integrationtests 34.49% <ø> (ø)
unittests 52.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@erka
erka force-pushed the rd/v2/openapi-followup branch from e73fc96 to d52ee92 Compare May 8, 2026 18:05
A better workaround for #5823 without dirty things in Flipt GRPC.

Signed-off-by: Roman Dmytrenko <[email protected]>
@erka
erka force-pushed the rd/v2/openapi-followup branch from d52ee92 to 767bd46 Compare May 9, 2026 10:13
@erka erka added the automerge Used by Kodiak bot to automerge PRs label May 9, 2026
@kodiakhq
kodiakhq Bot merged commit 8e94ba0 into v2 May 9, 2026
35 of 37 checks passed
@kodiakhq
kodiakhq Bot deleted the rd/v2/openapi-followup branch May 9, 2026 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Used by Kodiak bot to automerge PRs size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants