-
Notifications
You must be signed in to change notification settings - Fork 229
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
Upgrade Babel to v7 #392
Comments
Merged
mcollina
added a commit
that referenced
this issue
Dec 14, 2018
mcollina
added a commit
that referenced
this issue
Dec 16, 2018
* Updated to @babel/core@7, node 10.14.2 Fixes: #392 * use @babel/polyfill in the tests * Fixed missing babel deps * test passing on Node 8.
X-oss-byte
added a commit
to X-oss-byte/Readable-stream
that referenced
this issue
Sep 21, 2023
* perf fix for Duplex require replacement I was noticing that RegExps called from Module._resolveLookupPaths were getting to be the hottest symbol in some profiles. readable-stream seems to be requiring Duplex from inside functions. It would be faster to not do a require on each invocation of Readable, ReadableState, Writable, WritableState. * build based on v5.1.0 * bump process-nextick-args * 2.0.5 * fix node repo link * bump isarray to 1.x juliangruber/isarray@0.0.1...v1.0.0 * Update package.json desc.: iojs v2.x => Node.js * Make general updates to build script. * Make processFile() accept URL or local path. * Update Node version in README on build. * Define RegExp's up top. * build for 5.5.0, fix build scripts to deal with arrow functions amongst other things and browser build tweaks * initial build for v5.6.0 * undo the let usage * make sure const and let replacements are not in the middle of words * build based on 5.7.0 now with babel new build uses babel for arrow functions and const/let stuff also makes iThings allowed failures * remove "retain lines" * fix for sync write issues in older node * build for 5.8.0 * bump tape version (mainly to get the tests to rerun) * 2.0.6 * build for 5.9.1 * build for 5.9.1 * and fix build * test against microsoft edge (#194) * add passthrough env variable to allow testing of native streams, also adds coverage * 2.1.0 * fixes #205 reuse existing import of "stream" * 2.1.1 * fix browserify build * 2.1.2 * build for v6.1.0 plus fixes for assert and buffer changes * 2.1.3 * update npm ignore * try ngrok * 2.1.4 * re-enable iThing tests and update zuul * add node 6 * add back ngrok * reenable iThings * maybe older android? * nope minimum android is 3.0 * maybe older ie? * no older ie, maybe older iThings * nope * 6.2.1 (#218) build for 6.2.1 * rm unneeded test * deal with template strings * rewrite test for older node * remove ngrok * trailing comma, I'm done for the day * add sauce connect * patch for wonky setTimeout in older node * fix typo * rm sauce connect * Revert "rm sauce connect" This reverts commit c213f1e. * rm older ithing * build for 6.2.2 * add shorthand properties babel transform * fix for test that has for-of loop * iThings seem to be broken, giving up * back to local tunnel * and remove from .zuul * first stab at 6.3.1 * bufferlist * fix prepend * fix tests more * 2.1.5 * build based on 6.4.0 * Added @mcollina to the README. (#236) * first stab at 6.5.0 * rm unneded regex * inline isArray * build for 7.0.0 * bundle dependencies * build for 7.1.0 and remove documentation but include link to node docs * 2.2.0 * this is why we can't have nice things, fixes (yarnpkg/yarn#1774) * 2.2.1 * fix gulp weirdness * 2.2.2 * Added the governance template. * Updated after @williamkapke suggestions. * build for 7.3.0 (#251) * Exclude `stream` module in react-native (#258) * 2.2.3 * node v7.7.2 build (#262) * replace bind() * re-add WriteReq and CorkedRequest constructors * Updated to v7.7.2, refactored the build to not crawl The new build system uses the tarball from nodejs.org. * try updating zuul * just remove android * 2.2.4 * removed duplicate writereq. See nodejs/readable-stream@7af74b0#commitcomment-21310482 * Bumped v2.2.5. * Fully revert #255 as not released yet in core. * Added node v7 to .travis.yml * Updated to v7.7.3. * Bumped v2.2.6. * remove bizzare circular deps * 2 input files * fix typo * Maintain the same interface when using READABLE_STREAM=disable * Added -browser.js files for Duplex and Writable. * Bumped string_decoder to v1.0.0. * Avoid caret during installation This restores support for older versions of NPM. * Bumped v2.2.7 * Pull node 7.8.0. test passes w/ READABLE_STREAM=disable. * Bumped v2.2.8 * switch to simpler method of stream substitution * and build * use a browser sub for the streams vs event emitter switch * move the stream and stream-browser to somewhere better * remove react-native field * Bumped v2.2.9. * README: link to specific api docs about streams (#279) * use safe-buffer (#282) use safe-buffer * Updated to node v7.10.0. * Better .travis.yml * Disabled test/parallel/test-stream-unpipe-event.js in node 0.8. * Added node 8 to .travis.yml * Bumped v2.2.10. * Depend with ~ over SafeBuffer. * Test that we do not depend using ^, to support old npm. * Bumped v2.2.11. * updated package inherits from 2.0.1 to 2.0.3 (#291) * Build for node 8.1.0 * re-add WriteReq and CorkedRequest constructors * 2.3.0 * READABLE_STREAM=disable works also with writable.js * Import from Node 8.1.2. * Bumped v2.3.1 * Convert classes loosely * Forward-port of nodejs#13850 nodejs/node#13850 * Bumped v2.3.2. * Partially solves perf issues with the UInt8Array checks. Fixes nodejs/readable-stream#302 * Use instanceof check for Uint8Array. * Removed useless typeof check. * Updated from Node 8.1.3. * Bumped v2.3.3 * add lrlna to readme * built 8.9.4. * Working on old nodes * Updated README * Updated dependencies * Fix tests for Node master / citgm * Revert "Fix tests for Node master / citgm" This reverts commit 3dc883c734c7990f803ac05a8a64ce70d0815b1d. * Proper fix, soft-detecting there is a custom inspect. * removed saucelabs testing for now * Fix browser support for util.inspect.custom fix * Bumped v2.3.4 * Support for the Foundation ci. * Fixed broken lolex integration. * Bumped v2.3.5 * Fix lolex integration test on old nodes * Removed zuul as a dependency. * Bump string_decoder to versio 1.1.1 (#330) * Don't add the Array.prototype.forEach replacement to files that don't use forEach. (#331) * Updated to Node 8.11.1 (#332) * Bumped v2.3.6. * Built with 29 tests failing * 27 tests to go * Down do 20 failing tests * removed comment * 18 tests to go. * .npmignore: "docs/" => "doc/" to match real name (#342) * Updated to Node 10.5.0. 2 failures left to go. * all tests passing * Node.js 6 working. * Removed dependency on core-util-is * removed dependency on safe-buffer * only a single entry point * Integrate airtap (#345) * add airtap for continuous browser tests * ad-hoc fixes for browser tests * comment out failing browser tests * restore safari * Updated. * more airtap quirks * remove --no-coverage for airtap * better sauce credentials for travis * Maybe better env variables for airtap * removed sauce credentials from .travis.yml * added back all browsers * added back loopback host * Updated README links for sauce and travis * Suport for IE11 * README tweaks * Bumped v3.0.0-rc.1 * Removed babel-transform-runtime. Updated to Node 10.6.0. * Added back IE 11. * Bumped v3.0.0-rc.2 * Updated to Node 10.8.0. * Bumped v3.0.0-rc.3 * browsers tests should be passing again * Added a thanks note to Sauce Labs * Added usage section. * Removed inactive members (#351) * Added changelog of breaking changes * Bumped v3.0.0 Fixes #283 Fixes #284 Fixes #212 Fixes #297 Fixes #346 Fixes #339 Fixed #335 * Fix error messages (#352) * fix error messages to match node 10.8.0 * npm run update-browser-errors * test errors * Removed process-nexttick-args as it's not needed anymore (#354) * Removed process-nexttick-args as it's not used anymore * Rmoeved process-nexttick-args everywhere, it's not needed * Bumped v3.0.1. * Import Node 10.9.0. Redo the fix for util.inspect.custom (#357) * Import Node 10.9.0. Redo the fix for util.inspect.custom Fixes nodejs/readable-stream#355 * Removed util.inspect and util.inspect.custom poly in the browser test * fallback to 'inspect' if util.inspect.custom is not available * Bumped v3.0.2. * remove Buffer constructor usage (#358) (#359) * remove Buffer constructor usage (#358) * Replace occurrences of `new Buffer(...)` with `Buffer.alloc(...)` or `Buffer.from(...)`, as appropriate * squash! remove unused block * replace Buffer.alloc() with Buffer.from() where appropriate (#360) * Add node >= 6 to package.json (#362) * Updated to Node 10.10 (#363) * Updated to Node 10.10 * fix failing test, the code is correct * Bumped v3.0.3. * Update to node 10 11 (#366) * Fix errors in IE11 * Polyfill Number.isNaN * Don't create Symbol.asyncIterator if Symbol is not defined Fixes #364 * Updated to Node 10.11.0. * Fixed browsers tests * Bumped v3.0.4. * Emit no experimental warning in browser. (#367) closes #361 * Bumped v3.0.5. * Fixed build regexp for IE11. (#369) * Bumped v3.0.6 * Add build regexp IE11 (isInteger) (#389) * Ignore airtap and travis configs (#388) * Ignore airtap and travis configs * Remove .zuul.yml from ingore list * Updated to @babel/core@7, node 10.14.2 (#393) * Updated to @babel/core@7, node 10.14.2 Fixes: nodejs/readable-stream#392 * use @babel/polyfill in the tests * Fixed missing babel deps * test passing on Node 8. * Added yarn.lock to .gitignore * Bumped v3.1.0. * Use @babel/preset-env exclusively (#395) * Fix build script to support Windows * Use `@babel/preset-env` exclusively * Rebuild node 10.14.2 * Run update-browser-errors * Bumped v3.1.1. * export * from `stream` when `process.env.READABLE_STREAM === 'disable'` (#399) * Updated to v10.15.2 (#401) * Bumped v3.2.0. * Build from Node v10.15.3 (#402) * Bumped v3.3.0. * Travis CI: Remove the deprecated sudo tag (#405) * add missing exports for browsers (#409) * add missing exports for browsers * test-stream-finished in browser * test-stream-pipeline in browser * Add node v12 to .travis.yml (#410) * Add node v12 to .travis.yml * Updated to Node v10.15.3 * Remove error checks * Bumped v3.4.0. * build/test-replacements.js: fix typo. (#415) * doc: indicate stream impl from node versions (#424) * doc: indicate stream impl from node versions * doc: fix typo Co-Authored-By: Vincent Weevers <mail@vincentweevers.nl> Co-authored-by: Vincent Weevers <mail@vincentweevers.nl> * Update to Node v10.18.1 (#420) * Update to Node v10.17.0 * Browsers might not have Symbol, from edition * Use Node 12 for airtap * test passing on browser * Update to Node 10.18.1 * fixed file patterns for build * possibly browser fix * test passing down to Node 6 * Bumped v3.5.0 * [Fix] babel's "loose mode" class transform enbrittles BufferList (#428) ```js Object.defineProperty(Object.prototype, util.inspect.custom, { set(v) { throw 'this should not happen inside readable-stream'; } }); ``` With this change, I believe the output will use [[Define]] instead of [[Set]] for https://github.com/nodejs/node/blob/c101251a95cc82142bee4637f8db6cc360a06d82/lib/internal/streams/buffer_list.js#L167, and thus no longer fail when Object.prototype is modified. * Bumped v3.6.0 * Remove Calvin from the streams wg * Add ronag to the streams wg (#430) * Remove Irina and Yosh * Add vweevers to streams wg (#431) * ci: use the shared Travis config (#444) Closes #442 * tests: add GitHub Actions CI (#418) * tests: add GitHub Actions CI * fix: add missing newline at the end of ci.yml * fix: add `pull_request` trigger to GitHub Workflows Co-Authored-By: Michaël Zasso <targos@protonmail.com> * fix: add missing `npm install` * fix: add fail-fast: false to ci.yml Co-Authored-By: XhmikosR <xhmikosr@gmail.com> * fix: Split npm install and npm run test in ci.yml Co-authored-by: Michaël Zasso <targos@protonmail.com> Co-authored-by: XhmikosR <xhmikosr@gmail.com> * Upgrade airtap and use GitHub Actions (#443) * Upgrade airtap and use GitHub Actions - Upgrade airtap to v4 - Move Sauce Labs tests from Travis to GitHub Actions - Use playwright for headless local testing * Debug * Adds Node v14 to github actions (#445) * Cleanup CI (#446) Remove debug steps from sauce workflow (#443), remove travis, and replace badges in readme. * Add usage instructions for Webpack 5 (#455) * doc: make references to primary branch be main * Update to Node 18.0.0 (#471) * feat: Updated build system and removed legacy browsers compatibility. * feat: Update to Node 17.9.0. * feat: Updated to Node 18.0.0 and simplified errors and primordials. * test: Drop SauceLabs in favor of Playwright. * feat: Restore compatibility with Node 12.x. * fix: Fixed dependencies. * feat: Do not modify globalThis. * fix: Fix path on Windows. * fix: Fixed examples. * feat: Test bundlers. * fix: Fixed node bundle on older versions. * test: Test against browserify. * test: Test against webpack. * test: Improve build scripts. * test: Test against esbuild. * test: Fixed CI. * test: Do not use node: prefix. * test: Fix CI on Windows. * test: Fix ESBuild. * Evade override mistake (#474) * v4 launch details (#472) * fixed review Signed-off-by: Matteo Collina <hello@matteocollina.com> * Update README.md Co-authored-by: Vincent Weevers <mail@vincentweevers.nl> * Update README.md Co-authored-by: Vincent Weevers <mail@vincentweevers.nl> Co-authored-by: Vincent Weevers <mail@vincentweevers.nl> * Bumped v4.0.0 Signed-off-by: Matteo Collina <hello@matteocollina.com> * ci: add GitHub token permissions for workflows (#479) Signed-off-by: Varun Sharma <varunsh@stepsecurity.io> * Lazily require abort-controller (#478) * Lazily require abort-controller, since it throws when it has been bundled, then loaded in node * Make changes to build script and re-run them. Only did 18.0.0 to minimize the changeset. * Bumped v4.1.0 Signed-off-by: Matteo Collina <hello@matteocollina.com> * Update build.mjs (#484) * Remove requirement for bundler custom config (#489) * feat: use buffer, events, process dependencies and require it * fix: added missing process file for browserify bug workaround * fix: patch code after use strict * Update to Node v18.9.0 (#490) Signed-off-by: Matteo Collina <hello@matteocollina.com> Signed-off-by: Matteo Collina <hello@matteocollina.com> * Bumped v4.2.0 Signed-off-by: Matteo Collina <hello@matteocollina.com> * chore: just build to fix whitespace issues (#498) * fix: use workaround loading process-shim for browserify (#497) * Removed Numeric separator (#491) * Bumped v4.3.0 Signed-off-by: Matteo Collina <hello@matteocollina.com> * fix: removes `Readable.fromWeb` and `Readable.toWeb` methods (#506) * removing obsolete methods from Readable class Signed-off-by: Rishabh Bhandari <rishabhbhandari6@gmail.com> * moved the logic to replacements.mjs Signed-off-by: Rishabh Bhandari <rishabhbhandari6@gmail.com> * using the replacements.mjs logic Signed-off-by: Rishabh Bhandari <rishabhbhandari6@gmail.com> --------- Signed-off-by: Rishabh Bhandari <rishabhbhandari6@gmail.com> * Update to Node v18.16.0 (#515) * Update to Node v18.16.0 Signed-off-by: Matteo Collina <hello@matteocollina.com> * fixup Signed-off-by: Matteo Collina <hello@matteocollina.com> * fixup Signed-off-by: Matteo Collina <hello@matteocollina.com> * fixupo Signed-off-by: Matteo Collina <hello@matteocollina.com> * fixup Signed-off-by: Matteo Collina <hello@matteocollina.com> * format Signed-off-by: Matteo Collina <hello@matteocollina.com> --------- Signed-off-by: Matteo Collina <hello@matteocollina.com> * Bumped v4.4.0 Signed-off-by: Matteo Collina <hello@matteocollina.com> * chore: add node 20 to CI checks (#518) * fix: don't require `stream` package in codebase (#520) * fix: dont have require('stream') * chore: add stream importing regression tests to CI * chore: fix webpacking dist * chore: fix runner-prepare script * Bumped v4.4.1 Signed-off-by: Matteo Collina <hello@matteocollina.com> * fix: add string decoder as a dependency (#522) * Bumped v4.4.2 Signed-off-by: Matteo Collina <hello@matteocollina.com> --------- Signed-off-by: Matteo Collina <hello@matteocollina.com> Signed-off-by: Varun Sharma <varunsh@stepsecurity.io> Signed-off-by: Rishabh Bhandari <rishabhbhandari6@gmail.com> Co-authored-by: Sam Newman <sonewman@users.noreply.github.com> Co-authored-by: Ali Ijaz Sheikh <ofrobots@google.com> Co-authored-by: Calvin Metcalf <cmetcalf@appgeo.com> Co-authored-by: Calvin Metcalf <calvin.metcalf@gmail.com> Co-authored-by: Steve Mao <maochenyan@gmail.com> Co-authored-by: Shinnosuke Watanabe <snnskwtnb@gmail.com> Co-authored-by: Jesse McCarthy <git_commits@jessemccarthy.net> Co-authored-by: Matteo Collina <matteo.collina@gmail.com> Co-authored-by: Matteo Collina <hello@matteocollina.com> Co-authored-by: RangerMauve <RangerMauve@hotmail.com> Co-authored-by: Haroen Viaene <fingebimus@me.com> Co-authored-by: Esref Durna <edurna@gmail.com> Co-authored-by: Brian White <mscdex@mscdex.net> Co-authored-by: lrlna <shestak.irina@gmail.com> Co-authored-by: Moshe Kolodny <mkolodny@google.com> Co-authored-by: Rouven Weßling <me@rouvenwessling.de> Co-authored-by: Rick Waldron <waldron.rick@gmail.com> Co-authored-by: Vincent Weevers <mail@vincentweevers.nl> Co-authored-by: Gabriel "_|Nix|_" Schulhof <gabriel.schulhof@intel.com> Co-authored-by: Douglas Wilson <doug@somethingdoug.com> Co-authored-by: Jimmy Wärting <jimmy@warting.se> Co-authored-by: Julien Mounier <mounie_a@epita.fr> Co-authored-by: Steven <steven@ceriously.com> Co-authored-by: Paul Taylor <paul.e.taylor@me.com> Co-authored-by: cclauss <cclauss@me.com> Co-authored-by: Angelo Veltens <angelo.veltens@codecentric.de> Co-authored-by: XhmikosR <xhmikosr@gmail.com> Co-authored-by: kumavis <kumavis@users.noreply.github.com> Co-authored-by: Jordan Harband <ljharb@gmail.com> Co-authored-by: Robert Nagy <ronagy@icloud.com> Co-authored-by: Dominykas Blyžė <hello@dominykas.com> Co-authored-by: Tierney Cyren <accounts@bnb.im> Co-authored-by: Michaël Zasso <targos@protonmail.com> Co-authored-by: Michael Dawson <mdawson@devrus.com> Co-authored-by: Paolo Insogna <paolo@cowtech.it> Co-authored-by: Mark S. Miller <erights@users.noreply.github.com> Co-authored-by: Varun Sharma <varunsh@stepsecurity.io> Co-authored-by: Joe Hildebrand <joe+github@cursive.net> Co-authored-by: Colin Ihrig <cjihrig@gmail.com> Co-authored-by: Thomas Bergwinkl <bergos@users.noreply.github.com> Co-authored-by: jaxoncreed <jaxoncreed@gmail.com> Co-authored-by: Rishabh Bhandari <41220684+RishabhKodes@users.noreply.github.com> Co-authored-by: Jesse Wright <63333554+jeswr@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As suggested in pr #389 (https://github.com/nodejs/readable-stream/pull/389/files#r241132850). Used babel packages are old and can be upgraded.
It is actually not possible to build
readable-stream
without relying on alpha code ("babel-plugin-transform-optional-catch-binding": "^7.0.0-alpha.16"
)The text was updated successfully, but these errors were encountered: