MAIF / melusine

📧 Melusine: Use python to automatize your email processing workflow
https://maif.github.io/melusine
Other
352 stars 58 forks source link

Setup.py miss `melusine/config/config/config.json` #138

Closed remi-perrier closed 2 years ago

remi-perrier commented 2 years ago

When installing with pip, melusine/config/config/conf.json is missing.

image

Because glob in setup.py only capture JSON files in sub-folders of config, and skip JSON files in config folder itself.

Can be fixed by using glob.glob("melusine/config/**/*.json", recursive=True) instead of glob.glob("melusine/config/**/*.json"). I will create a pull request for this issue.

image

hugo-quantmetry commented 2 years ago

Well spotted ! Deltas between the pypi package and the github repo are tricky oO

We'll make the update