Add uri? predicate
Description
Environment
None
Attachments
5
Activity
David Nolen September 26, 2017 at 9:01 PM
David Nolen
September 26, 2017 at 9:01 PM
Sameer Rahmani September 25, 2017 at 3:48 PM
Sameer Rahmani
September 25, 2017 at 3:48 PM
patch updated
David Nolen September 25, 2017 at 7:37 AM
David Nolen
September 25, 2017 at 7:37 AM
Just require goog.Uri. You don't need to require goog.Uri in the tests.
Sameer Rahmani September 15, 2017 at 2:09 PM
Sameer Rahmani
September 15, 2017 at 2:09 PM
Patch Updated.
David Nolen September 15, 2017 at 1:33 PM
David Nolen
September 15, 2017 at 1:33 PM
I don't see why we need a try/catch there. We should remove that. Also don't we need to require goog.Uri?
Clojure introduced a `uri?` predicate in 1.9. Clojurescript currently doesn't have one.
@dnolen suggested on Slack that it would "probably need to tie to Closure Uri type".