Skip to content

Conversation

@JackPGreen
Copy link
Contributor

@JackPGreen JackPGreen commented Feb 11, 2025

We duplicate the specification of Python versions in each of the actions - instead we can centralise it to a single file that all actions can use.

Examples showing the version resolution (not necessarily the actions themselves!) working correctly:

Additional changes:

@JackPGreen JackPGreen changed the title Centralise Python version specification Centralise Python version specification in GitHub Actions Feb 11, 2025
@JackPGreen JackPGreen requested a review from yuce February 11, 2025 20:33
@JackPGreen JackPGreen self-assigned this Feb 11, 2025
@JackPGreen JackPGreen marked this pull request as ready for review February 11, 2025 20:36
@JackPGreen JackPGreen enabled auto-merge (squash) February 11, 2025 20:36
Copy link
Contributor

@yuce yuce left a comment

Choose a reason for hiding this comment

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

Just a single idea...

steps:
- id: extract-versions
run: |
python_versions='[ "3.8", "3.9", "3.10", "3.11", "3.12" ]'
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you also add 3.13 there?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I’ve no idea adding 3.13 - but could I raise a follow up PR to do that separately?

Copy link
Contributor

Choose a reason for hiding this comment

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

Sure.

@JackPGreen JackPGreen disabled auto-merge February 18, 2025 08:15
@JackPGreen JackPGreen merged commit 9b26a92 into hazelcast:master Feb 18, 2025
10 checks passed
@JackPGreen JackPGreen deleted the refactor-python-versions-declaration branch February 18, 2025 08:15
JackPGreen added a commit that referenced this pull request Feb 18, 2025
Addresses #719 (review)

Will also require updating branch protection rules.
JackPGreen added a commit to JackPGreen/hazelcast-python-client that referenced this pull request Feb 18, 2025
Addresses hazelcast#719 (review).

Will also require updating branch protection rules.
JackPGreen added a commit that referenced this pull request Feb 18, 2025
Addresses
#719 (review).

Will also require updating branch protection rules.
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