avirajkhare00 commented on issue #2776:
URL: https://github.com/apache/iggy/issues/2776#issuecomment-4064467868

   I would like to take this.
   
   I checked the repo layout first. There is already an established pattern for 
sync scripts in `scripts/ci/`, including `sync-rust-version.sh` and 
`python-version-sync.sh`, plus matching checks in 
`.github/workflows/_common.yml` and hooks in `.pre-commit-config.yaml`.
   
   My plan is:
   1. Add a dedicated `scripts/ci/sync-python-version.sh` script with `--check` 
and `--fix`, following the existing script style.
   2. Use a single source of truth for the Python interpreter version and 
validate or update the known drift points, including 
`foreign/python/Dockerfile.test`, `foreign/python/.devcontainer/Dockerfile`, 
and the `requires-python` declarations in the Python SDK, BDD, and examples 
projects.
   3. Wire the script into CI and pre-commit in the same way the existing sync 
scripts are handled.
   
   One extra constraint I will account for: the issue comment about wheel 
builds is valid, so I would also check the Python versions used in the wheel 
build workflow before finalizing the source of truth.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to