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

Space between "boundary" should be optional. #15

Merged
merged 1 commit into from Oct 6, 2013

Conversation

amitaibu
Copy link
Contributor

#14

@andrewrk
Copy link
Collaborator

thanks for this. if you add a test to make sure this behavior won't regress in the future I'll merge it pronto

@amitaibu
Copy link
Contributor Author

Sure. It's actually going to be my first nodejs test, so a pointer for an example, or where you want the test - would be appreciated ;)

@andrewrk
Copy link
Collaborator

The easiest thing would be probably to look at one of the standalone tests and use it as a template. then you can use npm test to run the test suite.

@amitaibu
Copy link
Contributor Author

amitaibu commented Oct 6, 2013

@ceoaliongroo added to my repo, and started writing the tests

@amitaibu
Copy link
Contributor Author

amitaibu commented Oct 6, 2013

@superjoe30 ,added a test, so it's ready for review.

@andrewrk andrewrk merged commit 8afb4f5 into pillarjs:master Oct 6, 2013
@andrewrk
Copy link
Collaborator

andrewrk commented Oct 6, 2013

thanks! this is released in 2.1.9.

@amitaibu amitaibu deleted the 14 branch October 15, 2013 14:01
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