Skip to content

WIP: Check if function exists before removing it#1255

Merged
nohwnd merged 2 commits intopester:masterfrom
renehernandez:mock-outside-describe
Mar 27, 2019
Merged

WIP: Check if function exists before removing it#1255
nohwnd merged 2 commits intopester:masterfrom
renehernandez:mock-outside-describe

Conversation

@renehernandez
Copy link
Copy Markdown
Contributor

1. General summary of the pull request

Fix #952

@renehernandez
Copy link
Copy Markdown
Contributor Author

@nohwnd Any updates on this? I am looking forward to getting it merge in :)

@nohwnd nohwnd merged commit b89f406 into pester:master Mar 27, 2019
@nohwnd
Copy link
Copy Markdown
Member

nohwnd commented Mar 27, 2019

@renehernandez I think I marked it as WIP because I saw that it's failing, and then did not check it again (which is why I add WIP to it, so I don't end up looking 20 times at the same PR trying to figure out what is the status). If you fix something next time please look if I did not mark it wip in the meantime, and ping me, it will get merged much faster :)

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.

Error when cleaning up Mocks defined outside of Describe, when running via Invoke-Pester

2 participants