Skip to content
This repository has been archived by the owner on Mar 13, 2024. It is now read-only.

Discussion: Linting auto installs using latest version of Python #155

Open
callumforrester opened this issue Oct 27, 2023 · 3 comments
Open

Comments

@callumforrester
Copy link
Contributor

The lint CI job automatically works with the latest version of Python. When a new version of Python is released, a projects particular dependencies may cause non-obvious.

At least 2 repositories in DiamondLightSource have been affected by a recent switch to Python3.12. See https://github.com/DiamondLightSource/mx_bluesky/actions/runs/6667521161/job/18122365120?pr=55.

The fix is to pin the Python version in .github/actions/install_requirements/action.yml.

Should we consider baking a known latest version into skeleton and manually updating/expecting projects to be updated by their maintainers?

@AlexanderWells-diamond
Copy link
Collaborator

I would prefer to see the Python version specified in the code.yml file - it's already an optional argument to the action. I suppose we could also pin the default at e.g. 3.11 or somesuch, but I think it's better to make it clearer in the top level CI file what version of Python will be used.

@DominicOram
Copy link
Contributor

I actually think we shouldn't pin it in the skeleton. If individual projects want to pin then that's fine but it's useful as a canary for later python versions. In fact, I think we should add a job that runs the tests against the latest python version too.

@DominicOram
Copy link
Contributor

If anyone comes here looking for a fix for the original issue: epics-base/p4p#126

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants