Upgrade python base version to python 3.10 about fastapi_best_architecture HOT 10 CLOSED

I am doing this work based on Python 3.8, but even if I add from __future__ Import annotations, I cannot use the | flag either

from fastapi_best_architecture.

Because this is not a library, it must pass local code testing, at least >= 3.7 or 3.8

from fastapi_best_architecture.

Hi, @downdawn

Is it convenient for you to perform the test?

I may lose my coding on May 1st because I don't have a laptop and I can only reply to messages via my phone

from fastapi_best_architecture.

Using the | symbol to separate is a feature of python3.10+

https://fastapi.tiangolo.com/python-types/#union

In the local win environment, I use conda to manage the switching environment, and in the Linux environment, I use pyenv. I can test different python versions.

from fastapi_best_architecture.

Yes, but the library based on 3.10 encoding can run in the lower version. I didn’t find the relevant PEP, so I‘m a little confused.

If just replace Optional to Union, it is very convenient, no-optional

from fastapi_best_architecture.

python3.10+ can run directly

def process_item(item: int | str):
    print(item, type(item))


if __name__ == '__main__':
    process_item(1)  # 1 <class 'int'>
    process_item("1")  # 1 <class 'str'>

python3.7-3.9 can import from __future__ import annotations run

from fastapi_best_architecture.

After testing, no_optiona can replace Optional[str] = None with Union[str, None] = None

Can be used in pre-commit.

from fastapi_best_architecture.

python3.10+ can run directly

python3.7-3.9 can import from __future__ import annotations run

Yes, This is expected.

There should be some problems in my submission that caused the error.

from fastapi_best_architecture.

After testing, no_optiona can replace Optional[str] = None with Union[str, None] = None

Can be used in pre-commit.

If we use PEP563, this scheme will not be needed. I have updated it to | through ruff.

from fastapi_best_architecture.

OK

from fastapi_best_architecture.