Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Invoking spago test does nothing (purs 0.13.8, spago 0.20.1) #789

Closed
emiel opened this issue May 6, 2021 · 5 comments · Fixed by #790
Closed

Invoking spago test does nothing (purs 0.13.8, spago 0.20.1) #789

emiel opened this issue May 6, 2021 · 5 comments · Fixed by #790
Labels

Comments

@emiel
Copy link

emiel commented May 6, 2021

Is spago 0.20.x supposed to work with PureScript 0.13.x or is it 0.14.x only? When invoking spago test with PureScript 0.13.8 and spago 0.20.1 nothing happens. I expect the build and test to run. I'm guessing this may somehow be related to psa but I'm unsure. Let me know if you need more info.

To reproduce:

Nothing happens when spago test invoked.

$ mkdir foo && cd foo
$ npm init -y
$ npm install --save purescript@0.13.8 spago@0.20.1
$ npx spago init
$ npx spago test
<<no output; no build>>

Expected output:

$ npm install --save spago@0.19.2
$ npx spago test
           Src   Lib   All
Warnings   0     0     0
Errors     0     0     0
[info] Build succeeded.
🍝
You should add some tests.
[info] Tests succeeded.
@f-f
Copy link
Member

f-f commented May 6, 2021

@emiel yep I can replicate this, and it's a regression. I'll look at it right away

@f-f
Copy link
Member

f-f commented May 6, 2021

Fix up in #790, I'll cut a release as soon as CI is done

@f-f f-f closed this as completed in #790 May 6, 2021
@f-f
Copy link
Member

f-f commented May 6, 2021

Released 0.20.2 with the fix

@emiel
Copy link
Author

emiel commented May 6, 2021

Wow @f-f! Thanks for the quick fix. I'm up and running again.

@f-f
Copy link
Member

f-f commented May 6, 2021

You're welcome 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants