Skip to content

Conversation

@bjlittle
Copy link
Member

@bjlittle bjlittle commented Aug 2, 2017

Closes #2706

@lbdreyer
Copy link
Member

lbdreyer commented Aug 3, 2017

If you say "Fixes #2706" somewhere in the PR description, I believe it automatically closes that issue when this PR get merged.
ALthough, maybe "closes #2706" works too?

@lbdreyer
Copy link
Member

lbdreyer commented Aug 3, 2017

Looking good @bjlittle !

@lbdreyer lbdreyer merged commit a340ab2 into SciTools:dask_mask_array Aug 3, 2017
@bjlittle
Copy link
Member Author

bjlittle commented Aug 3, 2017

@bjlittle
Copy link
Member Author

bjlittle commented Aug 3, 2017

Although it may need to be in a comment rather than the description ...

@bjlittle bjlittle deleted the dask-mask-remove-usages branch November 3, 2017 14:30
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.

3 participants