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

Deduplicate src-dirs #637

Open
drathier opened this issue Jul 24, 2024 · 2 comments
Open

Deduplicate src-dirs #637

drathier opened this issue Jul 24, 2024 · 2 comments

Comments

@drathier
Copy link
Contributor

I'd really like to deduplicate the test folders when they're exact matches; I have a project where I want to add the tests folder as an app src dir.

.concat(shouldAddTestsDirAsSource ? [testsDir] : []);

I'm getting this error if I add the tests folder to src-dirs:


/Users/drathier/drathier/elm-everything/tests/BasicsTests.elm

…matches more than one source directory:

/Users/drathier/drathier/elm-everything/tests
/Users/drathier/drathier/elm-everything/tests

Edit "source-directories" in your elm.json and try to make it so no source directory contains another source directory!

Note: The tests/ folder counts as a source directory too (even if it isn't listed in your elm.json)!```
@drathier
Copy link
Contributor Author

I bet https://stackoverflow.com/a/33121880/596041 would be enough

@lydell
Copy link
Collaborator

lydell commented Jul 24, 2024

I think it makes sense to not add the tests/ dir if it’s already in source-directories!

(Not sure about de-duplicating, though, I think it would make the most sense if elm-test and elm make gave the same error, instead of elm-test working while plain elm does not.)

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

No branches or pull requests

2 participants