search

wbolster / emacs-python-pytest

run pytest inside emacs
BSD 3-Clause "New" or "Revised" License
147 stars 25 forks source link

requires direnv #33

Closed flub closed 4 years ago

flub commented 4 years ago

When I have this in my startup file:

(require 'python-pytest)
(require 'direnv)

startup fails with:

Symbol's value as variable is void: direnv-non-file-modes

Switching the two lines around fixes this.

I guess you might as well just directly depend on direnv?

wbolster commented 4 years ago

yeah, it seems i broke this a few days ago without thinking it through: https://github.com/wbolster/emacs-python-pytest/commit/54f23a654bb502e54aa40d156d281f5902dc3fa1

but i reverted it yesterday: https://github.com/wbolster/emacs-python-pytest/commit/fc056faf2757c42641ed94d36a090e56eb13572f

i guess the latest melpa version should work for you?

flub commented 4 years ago

indeed, latest melpa doesn't have that issue anymore. thanks!

wbolster commented 4 years ago

thanks for reporting anyway, i didn't see this issue myself at first.