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

feat(specs): add transformation copilot to ingestion #3479

Merged
merged 5 commits into from
Aug 6, 2024

Conversation

Fluf22
Copy link
Contributor

@Fluf22 Fluf22 commented Aug 5, 2024

🧭 What and Why

🎟 JIRA Ticket: -

Changes included:

  • add copilot handling in ingestion transformation specs

@Fluf22 Fluf22 self-assigned this Aug 5, 2024
@Fluf22 Fluf22 requested a review from a team as a code owner August 5, 2024 14:51
@Fluf22 Fluf22 requested review from morganleroi and shortcuts August 5, 2024 14:51
@algolia-bot
Copy link
Collaborator

algolia-bot commented Aug 5, 2024

✔️ Code generated!

Name Link
🪓 Triggered by e7b223cafc9236098a4e9f2d54b1be6f3226b538
🍃 Generated commit 5d8061e67a092d16a181ffe0c9ec25d794ad577f
🌲 Generated branch generated/fix/ingestion-transformation-copilot
📊 Benchmark results

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

Language Rate
php 1385
javascript 1385
csharp 1101
java 921
ruby 888
swift 747
python 723
kotlin 463
go 460

@Fluf22 Fluf22 force-pushed the fix/ingestion-transformation-copilot branch from 2c8072c to fb86305 Compare August 5, 2024 14:52
@Fluf22 Fluf22 force-pushed the fix/ingestion-transformation-copilot branch from fb86305 to 9b9b070 Compare August 5, 2024 15:42
@Fluf22 Fluf22 requested a review from shortcuts August 5, 2024 15:48
Copy link

github-actions bot commented Aug 5, 2024

@Fluf22 Fluf22 force-pushed the fix/ingestion-transformation-copilot branch from 14a2b7f to d753283 Compare August 5, 2024 16:28
@Fluf22 Fluf22 force-pushed the fix/ingestion-transformation-copilot branch from d753283 to 8e768b7 Compare August 5, 2024 16:28
@Fluf22 Fluf22 requested a review from shortcuts August 5, 2024 16:30
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 great!

@@ -175,6 +175,8 @@ paths:
$ref: 'paths/transformations/transformationsTry.yml'
/1/transformations/search:
$ref: 'paths/transformations/transformationsSearch.yml'
/1/transformations/copilot:
Copy link
Member

Choose a reason for hiding this comment

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

we need to change that too D: I'll add a ticket

@Fluf22 Fluf22 merged commit ac0cadb into main Aug 6, 2024
21 checks passed
@Fluf22 Fluf22 deleted the fix/ingestion-transformation-copilot branch August 6, 2024 12:22
algolia-bot added a commit that referenced this pull request Aug 6, 2024
…ed) [skip ci]

Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-csharp that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-go that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-java that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-kotlin that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-php that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-python that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-ruby that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-scala that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-swift that referenced this pull request Aug 6, 2024
algolia/api-clients-automation#3479

Co-authored-by: algolia-bot <accounts+algolia-api-client-bot@algolia.com>
Co-authored-by: Thomas Raffray <Fluf22@users.noreply.github.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.

3 participants