remove redundant return values that exist for OO-style fluent programming
Closed this issue · 3 comments
I recently opened an SO question to understand this in the case of appendChild; it really seems like this should be modeled as returning Effect[Unit] (otherwise it is bit confusing and one has to use discard in some form). There may be other methods like this, but so far, this is the only one I've come across while writing my own code.
Sounds reasonable. I'll try to remember the suggestion next time there's a need to break some stuff!
I don't think 0.14 will necessarily require breaking any of the purescript-web packages, but it does break a few libraries which are quite low down in the ecosystem, so perhaps this might be a good time to do this?
We're about to update this library for the upcoming 0.14 release. I think this deserves to be a part of those changes.