Skip to content
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

Create MANIFEST.in #198

Closed
wants to merge 2 commits into from
Closed

Conversation

raybellwaves
Copy link

No description provided.

@Martinsos
Copy link
Owner

Hey @raybellwaves ! THanks for the PR, and my apologies for taking a look at it so ridiculously late. Could you give me a quick idea of why we should add MANIFEST.in, what is this solving?

@raybellwaves
Copy link
Author

Was recommended when creating a conda package

@Martinsos
Copy link
Owner

Got it, thanks! I do have MANIFEST.in in bindings/python, but here at the top level, I don't really understand why we should have it. And since this is the only request for it + not very detailed, I will be closing it, but we can reopen if it turns out there is a strong need for it.

@Martinsos Martinsos closed this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants