search

jfpedroza / neotest-elixir

Neotest adapter for Elixir
MIT License
38 stars 10 forks source link

feat: Specify full path for umbrella projects #30

Closed jfpedroza closed 7 months ago

jfpedroza commented 7 months ago

Closes #18

Hey @scottming, I'm implementing #18 to remove the warning about umbrella projects. I noticed for IEx you do a cd apps/child_app. Was that only for the warning, or does IExUnit require running in the child app? To know if we can remove the cd or not.

cc @fuelen, if you want to try this before merging.

fuelen commented 7 months ago

Just tried it, works fine in my tests

scottming commented 7 months ago

iExUnit requires running in the Child app.

jfpedroza commented 7 months ago

I won't remove that part then.

Merging.