Update Chapter 2 - Testing Code Using the Interactive Mode about purescript-book HOT 12 CLOSED

There's already an issue for this, but it doesn't quite address this particular problem: purescript/spago#550

from purescript-book.

This is apparently a bug in spago which should be fixed as of the next release.

from purescript-book.

To close this, we should update the repo to use that version of Spago in its instructions.

Do we need to specify specific tool versions?
Any reader who runs the suggested:

npm install -g spago

will get the latest version (0.15.2 currently).

https://book.purescript.org/chapter2.html#installing-tools

from purescript-book.

This is strange - an explicit spago install psci-support shouldn't be necessary, especially since psci-support already appears to be listed in the dependencies section in exercises/chapter2/spago.dhall.

from purescript-book.

So, funnily enough, if one just runs spago repl in a directory that doesn't have a spago.dhall file in it, this will work fine.

However, if one runs spago init and then spago repl, it will produce this error because a spago install was never ran. Thus, while it may be listed as a dependency, psci-support is never installed into that local folder, so that spago repl can function correctly.

from purescript-book.

Interesting, good find! Perhaps spago repl should ensure that all dependencies are installed before attempting to start the repl, then?

from purescript-book.

It should only if a spago.dhall file exists in the directory in which it runs. Otherwise, it should work as it currently does when there isn't such a file.

from purescript-book.

I opened purescript/spago#611

from purescript-book.

This bug no longer appears if one uses Spago v0.15.2.

To close this, we should update the repo to use that version of Spago in its instructions. However, the Spago release is a breaking change, so I don't know whether that would be problematic in other areas.

from purescript-book.

The only breaking change is the removal of an obscure CLI flag which I doubt this repo was using, so I think updating will be fine.

from purescript-book.

Yeah, I'd be tempted to just leave this. I think asking people to try updating their tools if things go wrong is the correct approach in general.

from purescript-book.

Do we need to specify specific tool versions?

At one point, I didn't specify the version of tools in my learning repo. Then, breaking releases were made that messed up a number of the commands I listed. So, I think it's better to list a specific tool version. That way, one can ensure that entire repo is updated before upgrading.

However, some tools (e.g. Spago) are more mature now than when I made that decision. So, I don't know whether that's really desirable at this point either.

Regardless, I don't think we should specify tool versions until it becomes an issue.

from purescript-book.