fix: type of arg type should be of type type

[yannforget]

When decorating a pipeline with @parameter, we use the type= argument to specify parameter type:

@pipeline(...)
@parameter(
    code="src_dhis2",
    type=DHIS2Connection,
    name="Source DHIS2 instance",
)
@parameter(
    code="dst_dataset",
    type=Dataset,
    name="Output dataset",
)
@parameter(
    code="org_units",
    type=str,
    multiple=True,
    name="Organisation units",
)
def my_pipeline():
   ...

We are not passing an instance of a class but the class/type itself. Therefore the annotated type of type should be of type type and not str | DHIS2Connection | Dataset | ....

[yolanfery]

type is too generic imo

I think this is correct instead, right ?

type: type[tr
    | int
    | bool
    | float
    | DHIS2Connection
    | IASOConnection
    | PostgreSQLConnection
    | GCSConnection
    | S3Connection
    | CustomConnection
    | Dataset],

[nazarfil]

When decorating a pipeline with @parameter, we use the type= argument to specify parameter type:

I think we need more context to understand the need for this change

• From the perspective of sdk itself it makes more sense to enforce typing to decrease number fo issues when pipeliens are built/executed and then na unkown error pops up, leading to a message in slack internal channel :D

[yannforget]

@yolanfery I didn't know about the type[] syntax, indeed that's a lot better!

@nazarfil I agree it's better to enforce typing, but the current one is wrong. E.g. we are not passing type=4 as the current annotations suggest but type=int if we want the parameter to be an int. And the type of int is type, not int

This means that you cannot check an OpenHEXA pipeline with a static type checker such as pyright/pylance/mypy