ayrat555 / mix.el Goto Github PK

Hello @ayrat555 ,

I've been involve the thread of the alchemist resurrection tonini/alchemist.el#371 , which I decided that I would fork the project and cut down some things.

Your package is a great way to interact with mix, the problem is that it clashes with the mix functionality of alchemist(now https://github.com/Sasanidas/Apprentice), so I'm starting to add your changes back to apprentice Sasanidas/Apprentice@e7166f0.

I hope that's good for you, and again, thanks for this amazing package.

Regards

First off: Great work, thanks!

I'd just wanted to ask what is needed to make errors openable in an umbrella app.
Since the compiler compiles apps recursively, the relative file paths in the output don't match in the root path of the umbrella application. The errors can be opened in compilation mode, but not found, due to the these paths. Any idea how to change that?

Currently, mix.el is not available through MELPA. If there is a demand for it, I'll upload the package there.

Please do, as installing from xELPA is a lot easier than from source when doing declarative package management on nixOS.

Currently this Emacs extension uses key bindings that are ill-advised by the Emacs conventions.

https://www.gnu.org/software/emacs/manual/html_node/elisp/Key-Binding-Conventions.html

Don’t define C-c letter as a key in Lisp programs. Sequences consisting of C-c and a letter (either upper or lower case; ASCII or non-ASCII) are reserved for users; they are the only sequences reserved for users, so do not block them.
Changing all the Emacs major modes to respect this convention was a lot of work; abandoning this convention would make that work go to waste, and inconvenience users. Please comply with it.

Please refrain from using the user space and instead use something in line with the conventions.

ps. Thanks for this package. Really like it but it did scratch my C-d ;-)

Hi, thanks for the package!

This is a feature request.

Sometimes I need to answer a prompt inside a Mix tasks, but mix-mode doesn't allow user input.

Most of the times there are flags to inhibit the prompt, so it is not bad, but it would be a nice feature to have.

Hi, I have an umbrella app, when running tests for an app within the umbrella, mix-test-current-buffer runs all the tests in the app, not just that file. After the test runs mix prints:

Paths given to "mix test" did not match any directory/file: test/path/to/file_test.exs

That's because it's missing the path prefix to go from the umbrella root to the sub-app in question.

When I find time I intend to look into this issue but I figured it's good to raise it here. Ah, I see this seems to be supported in apprentice: https://github.com/Sasanidas/Apprentice/blob/master/apprentice-mix.el#L52

Perhaps I should give that a go.

Thanks for this package BTW!