You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Bug: openjd check does not handle duplicate keys #125
Mmmm, that seems like an implementation detail rather than desired/defined behaviour. This makes me believe we need to do something here. Either update the spec to indicate duplicates do overrides, or update the spec and the library to consider these not valid job templates.
Looking at it, I'd consider it to be non-valid, since it's effectively dead code in a template and it won't ever be used, thus making it very misleading for users
Activity
crowecawcaw commentedon Mar 25, 2026
Python's YAML safe loader overrides duplicates, and the underlying library uses the safeloader here: https://github.com/OpenJobDescription/openjd-model-for-python/blob/mainline/src/openjd/model/_parse.py#L40-L46
Suggest closing as "won't fix".
epmog commentedon Mar 25, 2026
Mmmm, that seems like an implementation detail rather than desired/defined behaviour. This makes me believe we need to do something here. Either update the spec to indicate duplicates do overrides, or update the spec and the library to consider these not valid job templates.
Looking at it, I'd consider it to be non-valid, since it's effectively dead code in a template and it won't ever be used, thus making it very misleading for users