Skip to content

Use more modern streams2 streaming api#72

Open
leetreveil wants to merge 1 commit intomasterfrom
calippo
Open

Use more modern streams2 streaming api#72
leetreveil wants to merge 1 commit intomasterfrom
calippo

Conversation

@leetreveil
Copy link
Copy Markdown
Owner

No description provided.

test/test-id4.js Outdated
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 is actually the correct value for stik, the previous implementation was broken.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Best to check done event in all tests too.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better to call end when we can.

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.

1 participant