-
Notifications
You must be signed in to change notification settings - Fork 340
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
Clean up main namespace #718
Clean up main namespace #718
Conversation
Hello @StanczakDominik! Thanks for updating your pull request. Congratulations! There are no PEP8 issues in this pull request. 😸 Comment last updated at 2019-11-03 12:47:40 UTC |
Codecov Report
@@ Coverage Diff @@
## master #718 +/- ##
=========================================
+ Coverage 95.02% 95.63% +0.6%
=========================================
Files 57 55 -2
Lines 4722 4670 -52
=========================================
- Hits 4487 4466 -21
+ Misses 235 204 -31
Continue to review full report at Codecov.
|
Right, that should do it: [ins] In [3]: dir(plasmapy)
Out[3]:
['__builtins__',
'__cached__',
'__citation__',
'__doc__',
'__file__',
'__githash__',
'__loader__',
'__name__',
'__package__',
'__path__',
'__spec__',
'__version__',
'atomic',
'formulary',
'online_help',
'utils'] I'll still remove some unnecessary files from the main source directory. |
This was a packaging workaround that I'm very curious to see if it's breaking stuff again...
changelog/README.rst
for instructions, and if you need help with picking the PR type, ask!)docstrings. (Tests pop up at the bottom, in the checks box).
they're failing, ask!).