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

feat(action): use Python 3.12 #7

Merged
merged 1 commit into from
Oct 30, 2023
Merged

Conversation

felix11h
Copy link
Contributor

@felix11h felix11h commented Oct 30, 2023

Frontend pipelines are currently not running, see for example https://github.com/moneymeets/moneymeets-dashboard/actions/runs/6686275188/job/18165456270?pr=63. From the first look, this seems because of a Python version mismatch between what is used in the action.yml (3.11) and configured in pyproject.tom (~3.12).

I suggest fixing this for now by using 3.12 in the action, in order to unblock frontend as quickly as possible. In the next step, we can think whether we can use action-setup-poetry-python to automatically determine the Python version, like we have already prepared for action-merge-checks.

@felix11h felix11h merged commit 9c0974a into master Oct 30, 2023
4 checks passed
@felix11h felix11h deleted the feature/use-python-3-12-in-action branch October 30, 2023 08:10
@marns93
Copy link
Contributor

marns93 commented Nov 2, 2023

@felix11h I think we should switch to action-setup-poetry-python as we did for other ones. In this case we will auto-detect the python version for all repositories and with the next Python upgrade, those problems are gone.

@zyv
Copy link

zyv commented Nov 10, 2023

@felix11h I think we should switch to action-setup-poetry-python as we did for other ones. In this case we will auto-detect the python version for all repositories and with the next Python upgrade, those problems are gone.

👍 Are comments like that tracked as tickets? I'm afraid this stuff would get lost. Also for tickets one could enlist help from Frontend / Backend.

I think there was some other place also when we wanted to use our setup action with auto detection.

@marns93
Copy link
Contributor

marns93 commented Nov 13, 2023

@felix11h I think we should switch to action-setup-poetry-python as we did for other ones. In this case we will auto-detect the python version for all repositories and with the next Python upgrade, those problems are gone.

👍 Are comments like that tracked as tickets? I'm afraid this stuff would get lost. Also for tickets one could enlist help from Frontend / Backend.

I think there was some other place also when we wanted to use our setup action with auto detection.

https://moneymeets.atlassian.net/browse/MD-7058 :)

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

Successfully merging this pull request may close these issues.

4 participants