Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix readdirp usage #3

Closed
wants to merge 1 commit into from
Closed

Fix readdirp usage #3

wants to merge 1 commit into from

Conversation

ehmicky
Copy link
Contributor

@ehmicky ehmicky commented May 6, 2020

#2 introduced a bug due to readdirp returning an object, not a string. This PR fixes this.

@Tom-Bonnike
Copy link
Owner

Tom-Bonnike commented May 6, 2020

Thanks, I should have checked. 😄

Fixed in 79d81f7 already, closing this. 1.0.2 was published.

@Tom-Bonnike Tom-Bonnike closed this May 6, 2020
@Tom-Bonnike
Copy link
Owner

Tom-Bonnike commented May 6, 2020

I’ll write some tests if I ever need to make changes to this plugin in the future so that this doesn’t happen again, but leaving it as it is for now :)

@ehmicky
Copy link
Contributor Author

ehmicky commented May 6, 2020

Thanks! Sorry about introducing the bug.

@ehmicky ehmicky deleted the fix/readdirp-use branch May 6, 2020 16:46
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.

2 participants