Skip to content

Conversation

@pllim
Copy link
Member

@pllim pllim commented Jun 5, 2020

Is this what you meant in #32, @mhvk? It appears that pytest-mock was just added by @Cadair in #31.

@pllim pllim requested review from Cadair and mhvk June 5, 2020 15:24
@Cadair
Copy link
Member

Cadair commented Jun 5, 2020

:( but but why?

commented on #32

@Cadair
Copy link
Member

Cadair commented Jun 5, 2020

I am hard 👎 on this, this is a zero cost addition which helps sunpy out.

@bsipocz
Copy link
Member

bsipocz commented Jun 5, 2020

agree with @Cadair here. Unless there is a concrete issue that pytest-mock causes, I don't see why it should be removed.

@pllim
Copy link
Member Author

pllim commented Jun 6, 2020

Given that #32 is closed, I am closing this as well.

@pllim pllim closed this Jun 6, 2020
@pllim pllim deleted the rm-mock branch June 6, 2020 00:31
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