support sanctuary-show about list HOT 6 OPEN

Since this is specific for Sanctuary, what do you think about including it in sanctuary-list(*)?

(*) Which I have not forgotten about 😅

from list.

I like to imagine a future in which the Fantasy Land community at large adopts @@show. There's nothing Sanctuary-specific about the specification.

Edit: Updated link to point to correct project.

from list.

I don't see anything about @@show in the link? What is the difference between toString and @@show? If List implemented a toString method could it's @@show method simply be List.prototype["@@show"] = List.prototype.toString?

from list.

I don't see anything about @@show in the link?

Oops! I linked to the wrong project. I meant to link to sanctuary-show.

If List implemented a toString method could it's @@show method simply be List.prototype["@@show"] = List.prototype.toString?

If List#toString uses show to generate the string representations of its elements, then absolutely!

from list.

@davidchambers

If I understand correctly @@show should be implemented as:

List.prototype["@@show"] = function(l) {
  return "list(" + L.join(", ", L.map(show, l)) + ")";
}

However, that depends on the show function from sanctuary-show. I hope List will become a foundational library and therefore I'd like to avoid adding dependencies to it.

I understand that converting values to strings is used extensively in Sanctuary to do error reporting. But outside of that, I don't see many use cases for converting a list into an evaluateable string. Debugging and REPLs could be potential use cases but JavaScript has debuggers and REPLs that don't rely on converting values to strings.

Taking that into consideration, wouldn't it be more reasonable to add @@show in sanctuary-list?

from list.

I hope List will become a foundational library and therefore I'd like to avoid adding dependencies to it.

I hope that show will become foundational and that all ADTs will support it. ;)

Taking that into consideration, wouldn't it be more reasonable to add @@show in sanctuary-list?

It's certainly reasonable. It seems that Sanctuary is opinionated to the point that we need a sanctuary- package for everything (except Future a b).

from list.