search

sanctuary-js / sanctuary-def

Run-time type system for JavaScript
MIT License
294 stars 23 forks source link

documentation: remove ‘concat ({}) ({})’ to avoid confusion #194

Closed davidchambers closed 6 years ago

davidchambers commented 6 years ago

See #193

Commit message:

Although this naive concat function cannot operate on objects, Z.concat and S.concat can do so. concat ({}) ({}) is thus a confusing example: it is a logical operation but it is not supported due to a shortcoming. concat (null) (null) is clearer.

Do these changes remove some of the confusion, @jeanbaptiste-brasselet?