jfpedroza / neotest-elixir

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

Is this repository still being maintained? #26

Closed scottming closed 10 months ago

scottming commented 1 year ago

If you don't have much time, I'd happily become a maintainer.

I want to add more tests to make maintenance of this repository easier. Additionally, I would like to remove support for mix_test_interactive because neotest's watch feature is really great.

jfpedroza commented 1 year ago

Hey. As I replied in the email, I plan to resume work on this, but I can give you access to the repo.

I will give Neotest watch a try and see if mix_test_interactive can be removed.

scottming commented 1 year ago

As I replied in the email

Sorry, I missed that email. I saw it right now, thanks.

I plan to resume work on this

That's great.

but I can give you access to the repo.

Thanks.

I will give Neotest watch a try and see if mix_test_interactive can be removed.

One thing I want to mention is Neotest's watch needs the test code to rely on the product code directly.

For example:

alias ProductCodeModule

This way will watch the ProductCodeModule, but

module = ProductCodeModule
alias module

won't.

And I haven't test it with ElixirLS, but Lexical works well.

jfpedroza commented 10 months ago

I've invited you to the repo.

scottming commented 10 months ago

Thanks again.