weavejester / integrant

Micro-framework for data-driven architecture

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

valid-config-key? should include check that keyword has a namespace

RickMoynihan opened this issue · comments

I've just tried to put a spec on a datastructure that contains a system map, and thought I'd use the function ig/valid-config-key?, however it appears it doesn't test for the keyword also being a namespaced keyword.

Might it be worth tightening the test here?

(or (keyword? key) (composite-key? key)))

Yes, that sounds like a good idea.