Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace oneOf with anyOf in openapi schemas #100

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

nflaig
Copy link
Member

@nflaig nflaig commented Apr 29, 2024

Current openapi spec is invalid, see validation result.

See ethereum/beacon-APIs#421 for more rational.

@nflaig
Copy link
Member Author

nflaig commented Oct 9, 2024

would be good to get this in before the next builder spec release, this makes running spec tests more difficult as api spec is invalid and requires workarounds

Copy link
Member

@ralexstokes ralexstokes left a comment

Choose a reason for hiding this comment

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

interesting, if this works how I think it does then it makes sense!

thanks

@ralexstokes ralexstokes merged commit 91e2916 into ethereum:main Oct 10, 2024
3 checks passed
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