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(python): 3.8 compatibility and CI multi-version [skip-bc] #3613

Merged
merged 26 commits into from
Aug 29, 2024

Conversation

shortcuts
Copy link
Member

@shortcuts shortcuts commented Aug 29, 2024

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-2835 https://algolia.atlassian.net/browse/DI-2844

closes algolia/algoliasearch-client-python#570

Changes included:

update the ci matrix logic to support multi version testing, the goal is to only run relevant part (generation, bundling and cts) against every versions, but keep the most important checks (e.g. java bc, swift duplicates, push codegen, artifact storage) on the "main" version (the latest for python)

also refactor the login a bit in order to avoid spreading language-specific options

@shortcuts shortcuts self-assigned this Aug 29, 2024
@algolia-bot
Copy link
Collaborator

algolia-bot commented Aug 29, 2024

✔️ Code generated!

Name Link
🪓 Triggered by e3f2e0373dd409b6edffe91e782043b3c0092358
🍃 Generated commit 1569e5ba566c3743b40b41a6eae5b65fb9c16dd6
🌲 Generated branch generated/chore/python-multi-version-ci
📊 Benchmark results

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

Language Req/s
javascript 1680
php 1434
csharp 1175
python 986
java 966
ruby 875
swift 769
go 571
kotlin 439

Copy link

github-actions bot commented Aug 29, 2024

@shortcuts shortcuts marked this pull request as ready for review August 29, 2024 12:01
@shortcuts shortcuts requested a review from a team as a code owner August 29, 2024 12:01
@shortcuts shortcuts changed the title chore(ci): multi version build for python fix(python): 3.8 compatibility and CI multi-version Aug 29, 2024
@shortcuts shortcuts changed the title fix(python): 3.8 compatibility and CI multi-version fix(python): 3.8 compatibility and CI multi-version [skip-bc] Aug 29, 2024
@shortcuts shortcuts enabled auto-merge (squash) August 29, 2024 14:23
millotp
millotp previously approved these changes Aug 29, 2024
Copy link
Collaborator

@millotp millotp left a comment

Choose a reason for hiding this comment

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

sooooooo nice
best CI ever

@shortcuts shortcuts merged commit 036f9ac into main Aug 29, 2024
25 checks passed
@shortcuts shortcuts deleted the chore/python-multi-version-ci branch August 29, 2024 15:39
algolia-bot added a commit that referenced this pull request Aug 29, 2024
… (generated) [skip ci]

Co-authored-by: Clément Vannicatte <[email protected]>
Co-authored-by: Pierre Millot <[email protected]>
algolia-bot added a commit to algolia/algoliasearch-client-python that referenced this pull request Aug 29, 2024
…ted)

algolia/api-clients-automation#3613

Co-authored-by: algolia-bot <[email protected]>
Co-authored-by: Clément Vannicatte <[email protected]>
Co-authored-by: Pierre Millot <[email protected]>
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.

latest version only supports python 3.11
3 participants