Add type check for all the registry tags · feathr-ai/feathr#256

(8 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Scala (1.929 Stars) (244 Forks)batch import

good first issuehelp wanted

Beschreibung

Currently the registry tags are a dictionary format like this:

{"description":"this is a test feature"}

but user could easily do something else like

{"description", "this is a test feature"}

and the object will not be serialized as it's not a valid python dict. We should add a type check for that to avoid those kind of errors.

Tech Stack: python
Domain: backenddata
Issue Type: feature
Schwierigkeit: 2
Geschätzte Zeit: under 1 hour
Aktivitätsstatus: blocked
Klarheit: clear
Voraussetzungen: Basic Python
Einsteigerfreundlichkeit: 85
Research-Richtung: The registry tags are expected to be a Python dictionary. To add a type check, locate the code that processes registry tags (likely in a Python file related to feature definition or registry client). Implement a validation function that uses isinstance to ensure the tags input is a dict, and raise a TypeError or ValueError with a descriptive message if it is not. Ensure existing tests pass and consider adding a test for the invalid case.