Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

delete __init__.py from root #75

Open
epogrebnyak opened this issue Feb 23, 2021 · 2 comments
Open

delete __init__.py from root #75

epogrebnyak opened this issue Feb 23, 2021 · 2 comments

Comments

@epogrebnyak
Copy link

Does this https://github.com/domokane/FinancePy/blob/master/__init__.py have any purpose? Any imports depend on it? (we need to refactor them if they do)

@domokane
Copy link
Owner

The tests cases may need it.

@epogrebnyak
Copy link
Author

The tests cases may need it.

For pytest one might need package to be installed in editable mode with pip install -e ., poetry can enter the enviroment with poetry run pytest. We have to keep in mind this is a bit of red flag something need init.py in root, although not anything to shoot immediately.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants