Skip to content

Commit

Permalink
Fix/include requirements in manifest (#413)
Browse files Browse the repository at this point in the history
* Tests pass even when fire and numpydoc are not installed.

Import fire and numpydoc library lazily in cli.py. This way, there is
no error when in helper.py when these libraries are not installed.

* Include requirements* in MANIFEST.in.

Otherwise, install may fail because setup.py now reads from
requirements.txt.
  • Loading branch information
benjamin-work authored and ottonemo committed Dec 17, 2018
1 parent 08cfcc7 commit 0918820
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
2 changes: 2 additions & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

### Fixed

- Include requirements in MANIFEST.in


## [0.5.0] - 2018-12-13

Expand Down
1 change: 1 addition & 0 deletions MANIFEST.in
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
include VERSION
include LICENSE
include *.rst
include requirements*

0 comments on commit 0918820

Please sign in to comment.