Skip to content

does not support a dictionary of list of strings #21

Closed
@camerondavison

Description

@camerondavison

Currently normalize_dict blows up on the following if added the the tests

assert normalize_dict({"a": ["1", "2"]}) == {"a": ["1", "2"]}

Given the type signature it seems like this should be supported. It looks like since normalize_dict is deconstructing any lists with

        elif isinstance(val, list):
            val[:] = [normalize_dict(x) for x in val]

this assumption is that all elements in the list are also dictionaries. It seems like it should be okay to have a list of non dictionary items.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions