vim-turbux icon indicating copy to clipboard operation
vim-turbux copied to clipboard

[Feature Request] Support Elixir Umbrella Apps

Open joshuaballoch opened this issue 7 years ago • 3 comments

*FYI - I do not know if there's a recommended format for submitting an issue/feature request, so please LMK if I can update this.

I use vim-turbux with an elixir umbrella app. Currently vim-turbux supports mix tests, but a small modification needs to be made to support umbrella apps.

The current result of the turbux command is:

mix test test/path/to/test_file_test.exs

In an umbrella app, instead of tests being in the test directory, elixir apps are hosted in apps/app_name, so their tests are hosted in apps/app_name/test. The result of the turbux command is:

mix test apps/app_name/test/path/to/test_file_test.exs

I would like to modify the functionality to drop the initial apps/app_name from the path, so that the result reverts to the mix test test/path/to/test_file_test.exs.

I opened up the turbux code and am not sure how to yet, but I'll try to see if I can figure it out. Any tips would be appreciated!

joshuaballoch avatar Sep 15 '17 10:09 joshuaballoch

I'm certainly interesting in this kind of thing, but I confess I no next to nothing about Elixir, let alone the the ecosystem. If someone can come up with something that works, I'd gladly merge it.

jgdavey avatar Sep 18 '17 01:09 jgdavey

@jgdavey sorry for the late reply. Yeah that makes sense, I'd love to figure out something that works. I'm totally lost in the repo, though - never really tried writing or modifying a vim package before.

I was researching a little more and I know that one solution would be to use absolute paths rather than relative paths. Could you provide me any tips on either learning how to modify this repo, or on configuring whether this sends over absolute/relative paths?

joshuaballoch avatar Oct 12 '17 07:10 joshuaballoch

I took a stab at it. https://github.com/jgdavey/vim-turbux/pull/47

jgoerz avatar Apr 14 '19 02:04 jgoerz