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

fix(ruby): use discriminators for oneOf [skip-bc] #4310

Merged
merged 7 commits into from
Jan 6, 2025

Conversation

millotp
Copy link
Collaborator

@millotp millotp commented Jan 6, 2025

🧭 What and Why

🎟 JIRA Ticket:

fixes algolia/algoliasearch-client-ruby#507

Using discriminator to determine which class of oneOf to deserialize into.

Also cleanup some unused part of other templates.

@millotp millotp requested a review from a team as a code owner January 6, 2025 11:47
@millotp millotp self-assigned this Jan 6, 2025
@algolia-bot
Copy link
Collaborator

algolia-bot commented Jan 6, 2025

✔️ Code generated!

Name Link
🪓 Triggered by 13505646460ba4f8b51d3f0ab7d39b83fc3d2246
🍃 Generated commit f39d25b7faa188658d0d210aede27ac5fe9fc513
🌲 Generated branch generated/fix/ruby-discriminator
📊 Benchmark results

Benchmarks performed on the method using a mock server, the results might not reflect the real-world performance.

Language Req/s
go 1627
javascript 1615
php 1427
csharp 1310
java 1074
python 1043
ruby 792
swift 745

@millotp millotp changed the title fix(ruby): use discriminators for oneOf fix(ruby): use discriminators for oneOf [skip-bc] Jan 6, 2025
Copy link
Member

@shortcuts shortcuts left a comment

Choose a reason for hiding this comment

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

looks PERFECTO

@millotp millotp merged commit 653b4f3 into main Jan 6, 2025
28 checks passed
@millotp millotp deleted the fix/ruby-discriminator branch January 6, 2025 14:59
algolia-bot added a commit that referenced this pull request Jan 6, 2025
… [skip ci]

Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-ruby that referenced this pull request Jan 6, 2025
algolia/api-clients-automation#4310

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
algolia-bot added a commit to algolia/algoliasearch-client-scala that referenced this pull request Jan 6, 2025
algolia/api-clients-automation#4310

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
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.

Hits Array Contains Nil Values in client.get_recommendations Response
3 participants