build: add data dir to MANIFEST.in

balajtimate commented 9 months ago

Description

Add data/* to MANIFEST.in to include them in the build when using setuptools

Apparently this is also needed, including a __init__.py is not enough.

Type of change

[X] Bug fix (non-breaking change which fixes an issue)

Checklist

Please carefully read these items and tick them off if the statements are true or do not apply.

[X] I have performed a self-review of my own code
[X] My code follows the existing coding style, lints and generates no new warnings
[X] I have added type annotations to all function/method signatures, and I have added type annotations for any local variables that are non-trivial, potentially ambiguous or might otherwise benefit from explicit typing.
[X] I have commented my code in hard-to-understand areas
[X] I have added ["Google-style docstrings"] to all new modules, classes, methods/functions or updated previously existing ones
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes and I have not reduced the code coverage relative to the previous state
[X] I have updated any sections of the app's documentation that are affected by the proposed changes

If for some reason you are unable to tick off all boxes, please leave a comment explaining the issue you are facing so that we can work on it together.

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (a86cc09) 100.00% compared to head (fcc44dc) 100.00%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## dev #164 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 13 13 Lines 1143 1143 ========================================= Hits 1143 1143 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

balajtimate commented 9 months ago

Sorry, I forgot to include data in the MANIFEST in the previous PR, I already opened a PR for the bioconda recipe , if this one gets merged, I'll update the release and fix the test there.

zavolanlab / htsinfer