task: Ensure all Python components work with Python 3.12

[ogenstad]

Component

API Server / GraphQL, Git Integration

Task Description

No response

[ogenstad]

Pendulum 3.0 (https://github.com/sdispater/pendulum/releases/tag/3.0.0) has been released now which I think was the last blocker for Python 3.12

[gmazoyer]

It seems sync cannot be used with Python 3.12 due to its dagster dependency which is not 3.12 compliant yet (they even made sure to have pendulum < 3 as dep).

[dgarros]

thanks @gmazoyer, that was one of the main reason to separate it into a dedicated package
I think we can leave the sync plugin for now but it would be good to support 3,12 for the core if possible.

[ogenstad]

Ok, but if dagster doesn't work with pendulum 3 perhaps we need to handle pendulum differently in the SDK to ensure that the sync project works with the latest SDK version?

Were we using dagster or was that some test?

[dgarros]

Ok, but if dagster doesn't work with pendulum 3 perhaps we need to handle pendulum differently in the SDK to ensure that the sync project works with the latest SDK version?

Good point, it would be good to support both pendulum 2 and 3 if possible, we need to be as flexible as possible in the SDK regarding our dependencies.

Were we using dagster or was that some test?

It's mainly a test right now but there is a good chance we'll use it in the future

[ogenstad]

Based on that I think we should remove dagster for now and look at supporting both pendulum 2 and 3 when we move forward and need it. It seemed like very little had changed with regards to pendulum as seen from the outside. Also looks like they considered ditching pendulum but have been holding off Python 3.12 support for the same reason: dagster-io/dagster#17350

[gmazoyer]

Based on that I think we should remove dagster for now and look at supporting both pendulum 2 and 3 when we move forward and need it.

I was looking at doing just that. It seems to be easy enough to just remove dagster dependencies now without breaking things.

It seemed like very little had changed with regards to pendulum as seen from the outside.

From our code point of view, this is true. And it does not seem they introduce any breaking changes.

[gmazoyer]

Addressed with merge 8b23d22