Closed zeyus closed 1 year ago
Makes sense
On Mon, 17 Oct 2022 at 14.45, zeyus @.***> wrote:
@.**** commented on this pull request.
In pyproject.toml https://github.com/NLP4ALL/nlp4all/pull/14#discussion_r997014681:
@@ -0,0 +1,13 @@ +[tool.pytest.ini_options] +pythonpath = [
- "."
Well when you run something in Python, by default it will only look for modules (import x) in paths specified in the python path variable. When you want to make it look in the directory you're working with you need to specify . (the current directory) so we can say import nlp4all
— Reply to this email directly, view it on GitHub https://github.com/NLP4ALL/nlp4all/pull/14#discussion_r997014681, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFIHMR5FRXRCN7YINRRRBJDWDVC6JANCNFSM6AAAAAARGKBXZY . You are receiving this because your review was requested.Message ID: @.***>
-- Francisco Abenza +34682248610
Fixes #13
run_dev.py
for running while in development, making it easier in future to have separate production and dev environments (with dev needing testing, linting and debugging)requirements-dev.txt
for similar reasons, but while we're working we should now usepip install -r requirements-dev.txt
which has all the right stuff in it :D