Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Fix python version to 3.9 #328

Merged
merged 1 commit into from
Mar 29, 2023
Merged

Fix python version to 3.9 #328

merged 1 commit into from
Mar 29, 2023

Conversation

leshikus
Copy link
Contributor

@leshikus leshikus commented Mar 28, 2023

This is a workaround for #332 (it fixes development env; runtime env is still needed)

@leshikus leshikus force-pushed the lesh/fix-python-39 branch from a67179c to 8446eec Compare March 28, 2023 13:58
@Garra1980
Copy link
Contributor

try

  • python 3.9.16
    in yml file instead just
  • python 3.9

@leshikus leshikus force-pushed the lesh/fix-python-39 branch from eea90f8 to 3ae22d8 Compare March 29, 2023 12:04
@leshikus leshikus requested review from alexbaden and Garra1980 March 29, 2023 13:24
@alexbaden
Copy link
Contributor

Shouldn't we use python 3.9.* or equivalent so we will get security updates, etc without having to manually bump the minor version each time?

@Garra1980
Copy link
Contributor

Shouldn't we use python 3.9.* or equivalent so we will get security updates, etc without having to manually bump the minor version each time?

yes, I think we should try this

@kurapov-peter
Copy link
Contributor

What's the point of setting the version for our dev envs anyway?

@leshikus leshikus force-pushed the lesh/fix-python-39 branch from 3ae22d8 to 65a1cab Compare March 29, 2023 15:46
@leshikus leshikus force-pushed the lesh/fix-python-39 branch from 088990e to ca45751 Compare March 29, 2023 16:19
@leshikus
Copy link
Contributor Author

fixed to 3.9.*

@Garra1980
Copy link
Contributor

What's the point of setting the version for our dev envs anyway?

to sync with our benchmarking envs at least

@alexbaden alexbaden merged commit 49692cd into main Mar 29, 2023
@alexbaden alexbaden deleted the lesh/fix-python-39 branch March 29, 2023 19:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants