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

Lint on the fly doesn't work with elm-test #115

Open
opsb opened this issue Jan 22, 2017 · 4 comments
Open

Lint on the fly doesn't work with elm-test #115

opsb opened this issue Jan 22, 2017 · 4 comments

Comments

@opsb
Copy link

opsb commented Jan 22, 2017

By default the tests/elm-package.json points to the src directory in the parent project. This causes the warning

image

Is it possible to loosen the constraint to allow for this scenario or is there a fundamental issue?

@halohalospecial
Copy link
Collaborator

Hi! Currently, Lint On The Fly works by copying source .elm files to the work directory (which can be a temporary directory if not set). If a source directory is outside the project directory, scary things might happen like unintentionally modifying or deleting your other files. Suggestions on how to solve this are most welcome :)

@splodingsocks, can you still remember the specific issue that you encountered regarding this?

@mrmurphy
Copy link

Yeah, I believe the issue here is that elm-test's default configuration makes your test files act like a totally different Elm project (since it's got its own elm-package.json. So this isn't so much of an issue with elm-test alone, but an issue whenever you're trying to use one separate Elm project from within another.

I haven't used lint-on-the-fly for a long time for this reason. It'd be nice to have working, though!

@nickdima
Copy link

nickdima commented Dec 8, 2017

Someone has a solution for this yet?

@samgqroberts
Copy link

I've been able to work around this issue by collocating my tests with my main modules. (Obviously this is only if you are okay with doing that, I personally prefer it). I followed the conversation here to get my tests under src/ (and remove the tests/ dir altogether) rtfeldman/node-test-runner#241

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

5 participants