Skip to content
This repository has been archived by the owner on Feb 10, 2021. It is now read-only.

Add a basic manifest to aid in packaging #40

Merged
merged 1 commit into from
Nov 3, 2017

Conversation

jakirkham
Copy link
Member

Make sure the LICENSE and README are included. Also ensure any compiled Python code is not included.

Make sure the LICENSE and README are included. Also ensure any compiled
Python code is not included.
@jakirkham
Copy link
Member Author

Any thoughts on this?

@mrocklin
Copy link
Member

mrocklin commented Nov 3, 2017 via email

@mrocklin
Copy link
Member

mrocklin commented Nov 3, 2017 via email

@jakirkham
Copy link
Member Author

Make sense. Is alright if I merge small PRs like this at my own discretion then?

@mrocklin
Copy link
Member

mrocklin commented Nov 3, 2017 via email

@jakirkham
Copy link
Member Author

Great, thanks. 😄

@jakirkham jakirkham merged commit deb553a into dask:master Nov 3, 2017
@jakirkham jakirkham deleted the add_manifest branch November 3, 2017 17:54
@jakirkham
Copy link
Member Author

Would it be possible to disable merging with a merge commit and/or make merge squash the default for this repo?

@mrocklin
Copy link
Member

mrocklin commented Nov 3, 2017 via email

@jakirkham
Copy link
Member Author

So I lack the permissions to do this myself. Would you be able to change that or could I have permissions to do that?

@mrocklin
Copy link
Member

mrocklin commented Nov 3, 2017

I've disabled the merge-commit option. I'm not sure how to fine-tune permissions like this. There's always the option to give everyone admin privileges, which might be reasonable. That would probably require a larger conversation including more of the current core contributors.

@jakirkham
Copy link
Member Author

Thanks for doing that. No worries.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants