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

More test updates #190

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

More test updates #190

wants to merge 5 commits into from

Conversation

vmx
Copy link
Contributor

@vmx vmx commented Mar 2, 2018

I've updated some more tests. I didn't get as much done as I wanted (and I'm a week late), but it's still some progress.

BTW: I'm still committed to updating the tests over the coming weeks.

vmx added 4 commits March 2, 2018 14:41
The updated tests pass without any changes on the code.
Make sure that `Buffer.includes()` always return `true` if an
empty string is given as value.
@jessetane jessetane requested review from dcposch and removed request for dcposch March 3, 2018 21:56
@dcousens dcousens self-assigned this Oct 9, 2023
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.

None yet

2 participants