Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add else block to throw error from catch in removeSync() #408

Merged
merged 1 commit into from
Apr 25, 2017
Merged

Add else block to throw error from catch in removeSync() #408

merged 1 commit into from
Apr 25, 2017

Conversation

RyanZim
Copy link
Collaborator

@RyanZim RyanZim commented Apr 25, 2017

Fixes #407

@RyanZim RyanZim requested a review from jprichardson April 25, 2017 13:55
@RyanZim RyanZim mentioned this pull request Apr 25, 2017
Closed
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.09%) to 86.185% when pulling c179797 on rmSync-patch into 9f5a130 on master.

@jprichardson jprichardson merged commit acd8916 into master Apr 25, 2017
@RyanZim RyanZim deleted the rmSync-patch branch April 25, 2017 18:18
@dependencies dependencies bot mentioned this pull request Dec 1, 2017
Merged
@renovate renovate bot mentioned this pull request Feb 14, 2018
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jprichardson jprichardson Awaiting requested review from jprichardson

Assignees
No one assigned
Labels
bug feature-remove platform-windows
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@RyanZim @coveralls @jprichardson