Empty hierarchical

[cryogenian]

According to RFC3986 host = IP-literal / IPv4address / reg-name and reg-name = *( unreserved / pct-encoded / sub-delims ) and reg-name is used only for defining host.

@garyb please, please review

[garyb]

I'm not sure if this is the right way to handle this, wouldn't it be better to have an empty (Array (Tuple Host (Maybe Port))) instead?

You're totally right about the rule, but in general I think allowing empty string values isn't very "PureScripty", especially if it's a newtyped string.

I don't mind looking at this though, as there are two other parser issues I know of that need fixing for sure too, so I can do that at the same time.

[cryogenian]

Ok, switched to empty array.

[garyb]

👍

[garyb]

Can you squash before merging please?

[cryogenian]

Sure

[cryogenian]

Shall I release this as 0.2.2?

[garyb]

I'll open a PR for my other parser fix first actually. Should only take about 5 mins, I have the change already as part of a bigger thing, but the rest of it needs more work.