Skip to content

A bunch of fixes for openapi-changes#560

Merged
daveshanley merged 10 commits intomainfrom
changes-fixes
Apr 13, 2026
Merged

A bunch of fixes for openapi-changes#560
daveshanley merged 10 commits intomainfrom
changes-fixes

Conversation

@daveshanley
Copy link
Copy Markdown
Member

@daveshanley daveshanley commented Apr 13, 2026

And vacuum!

A narrow fix. The following is an equivalent. only checking a 3.1 scalar union.

  - type: [null, string, number]
  - anyOf: [{type: null}, {type: string}, {type: number}]
Examples are not breaking changes, not ever.
Upgraded extension handling.
trying to not incorrectly mutate the bundling output.
This has already been fixed but now ordering is also irrelevent.
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.68%. Comparing base (78fa820) to head (42a6415).
⚠️ Report is 11 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##             main     #560    +/-   ##
========================================
  Coverage   99.68%   99.68%            
========================================
  Files         261      261            
  Lines       28982    29615   +633     
========================================
+ Hits        28890    29523   +633     
  Misses         55       55            
  Partials       37       37            
Flag Coverage Δ
unittests 99.68% <100.00%> (+<0.01%) ⬆️

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.

@daveshanley daveshanley merged commit e15ee6d into main Apr 13, 2026
6 of 7 checks passed
@blacksmith-sh
Copy link
Copy Markdown
Contributor

blacksmith-sh Bot commented Apr 13, 2026

Found 3 test failures on Blacksmith runners:

Failures

Test View Logs
github.com/pb33f/libopenapi/bundler/TestRenderInline_DigitalOceanAsync View Logs
github.com/pb33f/libopenapi/datamodel/low/v2/TestRolodexRemoteFileSystem View Logs
github.com/pb33f/libopenapi/datamodel/low/v3/TestRolodexRemoteFileSystem View Logs

Fix in Cursor

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