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

Mostly streams3 updates (should be v12) #1413

Merged
9 commits merged into from Oct 26, 2015
Merged

Mostly streams3 updates (should be v12) #1413

9 commits merged into from Oct 26, 2015

Conversation

zertosh
Copy link
Member

@zertosh zertosh commented Oct 26, 2015

The commits are pretty self-explanatory.

For the changelog:

#12.0.0

Node support changes: now testing against the latest node (currently 4.2.1). node 0.8 is no longer supported, and iojs isn't actively tested.

Stream3 everywhere! Everything has been updated to use streams3.

Sourcemaps charset now uses an `=` instead of a `:`. This fixes certain issues with Chinese characters in sourcemaps. See #753.

module-deps has been updated to fix root transforms on symlinked modules. See https://github.com/substack/module-deps/pull/99.

stream-http, the module that provides `http` support, dropped IE8 support. If you depend on this, see https://github.com/jhiesey/stream-http#ie8-note.

Removed `builtins` and `commondir` – both unused dependencies.

@zertosh
Copy link
Member Author

zertosh commented Oct 26, 2015

Also merge in: #1392

"has": "^1.0.0",
"htmlescape": "^1.1.0",
"stream-http": "^1.2.0",
"stream-http": "~2.0.0",
Copy link
Member

Choose a reason for hiding this comment

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

Can we change this to ^? @jhiesey's following semver.

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

These mostly include updates for newer versions of node. stream-http dropped IE8 support - needs shims to work (see #1384).
@feross
Copy link
Member

feross commented Oct 26, 2015

LGTM if all the tests pass

@ghost
Copy link

ghost commented Oct 26, 2015

Merged, thanks!

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants