borkdude / speculative

Unofficial community-driven specs for clojure.core
Eclipse Public License 1.0
188 stars 16 forks source link

Flatten #177

Closed bonega closed 5 years ago

bonega commented 5 years ago

Not really that valuable spec. Perhaps of use in a future with a context-aware IDE

borkdude commented 5 years ago

@bonega Thanks for your contribution. According to the docs of flatten it takes a nested sequential thing, so I think we should not accept ::ss/any, but (s/nilable :ss:/sequential). E.g. I don't want to see flatten in these results: https://re-find.it/?args=1.

As for the test: the other tests follow the pattern:

Can you complete the test like this? I'll document this pattern in the style guide.