Closed econchick closed 8 years ago
LGTM: makes me a little nervous to implement a pattern matching like specifically this with re, but I guess this is good enough if this is only used in the uri params stuff
Thanks @tardyp ! Can you explain a little bit in more detail what your hesitation is with the pattern matching?
Current coverage is
97.36%