-
Notifications
You must be signed in to change notification settings - Fork 131
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
Add "sources" config key #173
Comments
+1 |
Related to this: I just discovered that I'll add a "bug" label because I think the fix for this issue should also address it. |
Perhaps this can also be disabled somehow, since in the case where one's tests has additional dependencies, it should be structured as a separate project described in https://github.com/spacchetti/spago/#separate-devdependencies-or-test-dependencies ? |
@bbarker Once we enable this you'd explicitly have the globs in your { name = "my-tests"
, packages = ../packages.dhall
, dependencies = [ "console", "effect", "prelude" ]
, sources = [ "test/**/*.purs" ]
} |
Spago includes some default path globs in the build:
src/**/*.purs
test/**/*.purs
However, if you don't have those directories, it will complain:
I propose we instead make this explicit (and configurable) by adding a
sources
(orpaths
) key to the configuration, and removing the implicit paths.We should migrate existing configurations so the change is backwards compatible.
The text was updated successfully, but these errors were encountered: