Skip to content
This repository has been archived by the owner on Aug 5, 2019. It is now read-only.

Add missing __init__.py #3

Merged
merged 1 commit into from
Apr 28, 2018
Merged

Add missing __init__.py #3

merged 1 commit into from
Apr 28, 2018

Conversation

madisvain
Copy link
Contributor

The init.py files are required to make Python treat the directories as containing packages; this is done to prevent directories with a common name, such as string, from unintentionally hiding valid modules that occur later (deeper) on the module search path. In the simplest case, init.py can just be an empty file, but it can also execute initialization code for the package or set the all variable, described later.

…es as containing packages; this is done to prevent directories with a common name, such as string, from unintentionally hiding valid modules that occur later (deeper) on the module search path. In the simplest case, __init__.py can just be an empty file, but it can also execute initialization code for the package or set the __all__ variable, described later.
@madisvain
Copy link
Contributor Author

Hei @jaydensmith could you please merge and release. Missing this file seems to make this package behave in unexpected ways. We had no issues with Django itself but Celery tasks were not loading correctly because of this file missing.

This is a required file and would make it a proper package.

If there is anything I can do to help please let me know.

@jaydensmith jaydensmith merged commit 483bcbe into jaydensmith:master Apr 28, 2018
@jaydensmith
Copy link
Owner

No worries, I've merged it now.

@madisvain
Copy link
Contributor Author

@jaydensmith the release actually had no changes. Not sure if github removed this empty file but init.py is still not there even though the pull requested added it.

Could you add it and make a release once more please? Might be that github is behaving in unexpected ways here.

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

Successfully merging this pull request may close these issues.

None yet

2 participants