search

cmu-mars / brasscomms

0 stars 1 forks source link

better treatment of constants (strings in particular) #21

Closed ivoysey closed 7 years ago

ivoysey commented 7 years ago

maybe use https://pypi.python.org/pypi/attrs for parsing the config

ivoysey commented 7 years ago

make an enum for the end points? i think you can have enums indexed by strings, so that would give us "static" checking that we're using the right strings for those anyway.

ivoysey commented 7 years ago

there's always room for improvement on this kind of thing, but my commits from today go a long way. as long as we're more principled going forward, we should be OK, so i'm going to close this out.