-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add missing yaml file to package #118
Conversation
I tried to add that and it didn't work.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
For some reason, this doesn't work on my machine though, but it should... I'll try some more.
MANIFEST.in
Outdated
@@ -9,7 +9,7 @@ include requirements_docs.txt | |||
|
|||
recursive-include figanos *.json *.mplstyle *.py *.txt | |||
recursive-include docs *.rst conf.py Makefile make.bat *.jpg *.png *.gif | |||
recursive-include data *.json *.txt | |||
recursive-include data *.json *.txt *.yml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
recursive-include data *.json *.txt *.yml | |
recursive-include data *.json *.txt *.yml *.png |
Shouldn't we want to add the logo too ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question. I think we should opt for a different solution. I can probably suggest an implementation tomorrow. See: #115
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ça marche, merci!
Pull Request Checklist:
number
) and pull request (:pull:number
) has been added.What kind of change does this PR introduce?
*.yml
)Does this PR introduce a breaking change?
No.
Other information: