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

Yarn fails on initial npm install (as of 25 minutes ago) #1774

Closed
dbashford opened this issue Nov 10, 2016 · 40 comments
Closed

Yarn fails on initial npm install (as of 25 minutes ago) #1774

dbashford opened this issue Nov 10, 2016 · 40 comments

Comments

@dbashford
Copy link

Do you want to request a feature or report a bug?

Bug

What is the current behavior?

Yarn not working at all.

This is happening in both my build and my local environment as of 25 minutes ago, which is also is when readable-stream was last published. Note the error is occurring in readable-stream. Haven't debugged beyond noticing that coincidence.

If the current behavior is a bug, please provide the steps to reproduce.

npm install -g yarn
yarn --version

Error: ENOENT: no such file or directory, open '/Users/bashfd/.nvm/versions/node/v6.3.1/lib/node_modules/yarn/node_modules/buffer-shims/index.js'
    at Error (native)
    at Object.fs.openSync (fs.js:640:18)
    at Object.fs.readFileSync (fs.js:508:33)
    at Object.Module._extensions..js (module.js:549:20)
    at Module.load (module.js:458:32)
    at tryModuleLoad (module.js:417:12)
    at Function.Module._load (module.js:409:3)
    at Module.require (module.js:468:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/Users/bashfd/.nvm/versions/node/v6.3.1/lib/node_modules/yarn/node_modules/readable-stream/lib/_stream_readable.js:40:18)

What is the expected behavior?
Not error out 😄

Please mention your node.js, yarn and operating system version.
6.3.1
0.16.1
El Capitan

@dbashford dbashford changed the title Yarn fails on initial NPM install (as of 25 minutes ago) Yarn fails on initial npm install (as of 25 minutes ago) Nov 10, 2016
@jaxgeller
Copy link

The newest readable stream was built for node 7 https://github.com/nodejs/readable-stream/releases/tag/v2.2.0

@dbashford
Copy link
Author

So either that ought to have been a 3.0.0 release, or tar-stream, which is Yarn's dependency, ought to be pinned back at 2.1.x? Or something else?

cc @calvinmetcalf

@wyze
Copy link
Member

wyze commented Nov 10, 2016

Just tested this out and I was able to run yarn add readable-stream successfully. Are you doing an add or install? Can you provide your package.json or a modified one that can be used to reproduce this?

@dbashford
Copy link
Author

dbashford commented Nov 10, 2016

See initial issue above. Just installing yarn and running yarn --version.

Which is to say, this isn't about installing readable-stream, its about using Yarn to do anything after installing it from NPM in the last hour.

@sejoker
Copy link

sejoker commented Nov 10, 2016

Same issue
node 4.6.2
ubuntu 14
yarn 0.16.1
npm 3.10.9

@calvinmetcalf
Copy link

@jaxgeller I built it in node 6 just based on node 7 so that's not the issue

@dbashford what version of npm ?

@wyze
Copy link
Member

wyze commented Nov 10, 2016

My mistake @dbashford! I see now.

@dbashford
Copy link
Author

@calvinmetcalf 3.10.3

@jaxgeller
Copy link

is there a way to pin readable-stream@2.1.5 for yarn globally? (installing yarn thru npm install yarn -g

@calvinmetcalf
Copy link

I just did npm install -g yarn && yarn --version successfully

@josegonzalez
Copy link

josegonzalez commented Nov 10, 2016

@calvinmetcalf that works fine because it doesn't actually invoke readable-stream from tar-stream. Once you try yarn install with those dependencies, you should see the error (at least in our case).

@calvinmetcalf
Copy link

just did no error

@dbashford
Copy link
Author

Just yarn --version for me, happening both locally and in my Jenkins environment which checks the version before deciding whether or not to install an update.

@jaxgeller
Copy link

jaxgeller commented Nov 10, 2016

@calvinmetcalf what node version? for example, 6.3.0 works fine, 6.9.1 does not

@calvinmetcalf
Copy link

the error seems to be that bufer shim is is being loaded from

yarn/node_modules/buffer-shims/index.js

where it used to be before 2.2.0 instead of from

yarn/node_modules/readable-stream/node_modules/buffer-shims/index.js

where it is now making me wonder if there is some sort of cache issue ...

@calvinmetcalf
Copy link

@jaxgeller v6.9.1 for node, 3.10.7 for npm, that being said I'm using nave not nvm to manage versions, I wonder if it's an nvm issue

@mandragorn
Copy link

I'm getting the same issue across all build machines in my organization. If there is more info I can provide or any way I can help, let me know.

I'll be investigating workaround hacks to get things moving here until there is a fix or more acceptable workaround.

@mandragorn
Copy link

node: v4.6.1
npm: 3.10.9

@calvinmetcalf
Copy link

*sigh* I think I need to remove the bundled dependencies from readable stream and publish 2.2.1, apparently they aren't actually supported everywhere.

@josegonzalez
Copy link

@calvinmetcalf @jaxgeller and I have node v6.9.1 and npm 3.10.8, installed from the nodesource packages.

@dbashford
Copy link
Author

@calvinmetcalf thanks for jumping into this so quick! Much appreciated. 👏

@mandragorn
Copy link

@calvinmetcalf thank you for your quick update! You've saved me from a long day of trying to hack together a workaround for this.

@calvinmetcalf
Copy link

done v2.2.1, lesson learned today is bundled dependencies don't actually work and/or is a semver major change in the package

@wyze
Copy link
Member

wyze commented Nov 10, 2016

readable-streams@2.2.1 is published. Just reinstalled yarn and it is working properly. Thanks for the quick turnaround @calvinmetcalf!

@wyze wyze closed this as completed Nov 10, 2016
@dbashford
Copy link
Author

Tested and works. Thanks again @calvinmetcalf!

@josegonzalez
Copy link

josegonzalez commented Nov 10, 2016

We tested by doing the following (as root):

npm uninstall yarn -g
npm install yarn@0.16.01 -g

Seems to work, thanks @calvinmetcalf. If you're ever in NYC, hit up @jaxgeller for a beer/milkshake :)

@mandragorn
Copy link

Working here to. Thanks @calvinmetcalf !

@henrik242
Copy link

henrik242 commented Dec 22, 2016

@calvinmetcalf Suddenly got the same error :(

build	22-Dec-2016 12:56:41	Task :yarn actions class loader hash: 60f5f4117b1c68b0177ed15b33205b50
build	22-Dec-2016 12:56:41	Executing task ':yarn' (up-to-date check took 0.0 secs) due to:
build	22-Dec-2016 12:56:41	  No history is available.
build	22-Dec-2016 12:56:41	Starting process 'command '/mypath/build/nodejs/node-v6.9.2-linux-x64/bin/node''. Working directory: /mypath Command: /mypath/build/nodejs/node-v6.9.2-linux-x64/bin/node /mypath/build/yarn/node_modules/yarn/bin/yarn.js 
build	22-Dec-2016 12:56:41	Successfully started process 'command '/mypath/build/nodejs/node-v6.9.2-linux-x64/bin/node''
build	22-Dec-2016 12:56:41	/mypath/build/yarn/node_modules/yarn/bin/yarn.js:62
build	22-Dec-2016 12:56:41	      throw err;
build	22-Dec-2016 12:56:41	      ^
build	22-Dec-2016 12:56:41	
build	22-Dec-2016 12:56:41	Error: ENOENT: no such file or directory, open '/mypath/build/yarn/node_modules/bl/node_modules/readable-stream/duplex.js'
build	22-Dec-2016 12:56:41	    at Error (native)
build	22-Dec-2016 12:56:41	    at Object.fs.openSync (fs.js:640:18)
build	22-Dec-2016 12:56:41	    at Object.fs.readFileSync (fs.js:508:33)
build	22-Dec-2016 12:56:41	    at Object.Module._extensions..js (module.js:578:20)
build	22-Dec-2016 12:56:41	    at Module.load (module.js:487:32)
build	22-Dec-2016 12:56:41	    at tryModuleLoad (module.js:446:12)
build	22-Dec-2016 12:56:41	    at Function.Module._load (module.js:438:3)
build	22-Dec-2016 12:56:41	    at Module.require (module.js:497:17)
build	22-Dec-2016 12:56:41	    at require (internal/module.js:20:19)
build	22-Dec-2016 12:56:41	    at Object.<anonymous> (/mypath/build/yarn/node_modules/bl/bl.js:1:82)
build	22-Dec-2016 12:56:41	:yarn FAILED
build	22-Dec-2016 12:56:41	:yarn (Thread[Daemon worker Thread 10,5,main]) completed. Took 0.229 secs.
build	22-Dec-2016 12:56:41	
build	22-Dec-2016 12:56:41	FAILURE: Build failed with an exception.

yarn 0.17.10

Dependency path:

build	22-Dec-2016 12:56:41	  +-- tar-stream@1.5.2 
build	22-Dec-2016 12:56:41	  | +-- bl@1.2.0 
build	22-Dec-2016 12:56:41	  | +-- end-of-stream@1.1.0 
build	22-Dec-2016 12:56:41	  | | `-- once@1.3.3 
build	22-Dec-2016 12:56:41	  | +-- readable-stream@2.2.2 
build	22-Dec-2016 12:56:41	  | | +-- buffer-shims@1.0.0 
build	22-Dec-2016 12:56:41	  | | +-- core-util-is@1.0.2 
build	22-Dec-2016 12:56:41	  | | +-- isarray@1.0.0 
build	22-Dec-2016 12:56:41	  | | +-- process-nextick-args@1.0.7 
build	22-Dec-2016 12:56:41	  | | +-- string_decoder@0.10.31 
build	22-Dec-2016 12:56:41	  | | `-- util-deprecate@1.0.2 
build	22-Dec-2016 12:56:41	  | `-- xtend@4.0.1 

It seems that bl@1.2.0 was released 5 hours ago, which is about when our build started failing. 1.1.2 seems to work.

@jboelen
Copy link

jboelen commented Dec 22, 2016

Same issue as @henrik242. Things were working just fine yesterday.

yarn install
~/.nvm/versions/node/v6.9.1/lib/node_modules/yarn/bin/yarn.js:62
      throw err;
      ^

Error: ENOENT: no such file or directory, open '~/.nvm/versions/node/v6.9.1/lib/node_modules/yarn/node_modules/bl/node_modules/readable-stream/duplex.js'
    at Error (native)
    at Object.fs.openSync (fs.js:640:18)
    at Object.fs.readFileSync (fs.js:508:33)
    at Object.Module._extensions..js (module.js:578:20)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (~/.nvm/versions/node/v6.9.1/lib/node_modules/yarn/node_modules/bl/bl.js:1:82)

Temporary solution was to install it through brew on macOS

@calvinmetcalf
Copy link

so definitely not my fault this time as I didn't publish an update to readable-stream, @mcollina did publish a new version of bl, but that doesn't have any bundled dependencies so this is likely a bug in yarns cache resolution algorithm

@mcollina
Copy link

I did release a new version of bl. That had a new version on readable-stream. Definitely a resolution algorithm problem.

@calvinmetcalf
Copy link

calvinmetcalf commented Dec 22, 2016 via email

@evenstensberg
Copy link

CC @kittens @thejameskyle

@sorenvind
Copy link

For anybody finding this: We hit this bug today with version 0.17.10. Bumping the version to 0.18.1 solved the issue (for now).

@mynnx
Copy link

mynnx commented Dec 29, 2016

Oddly enough, @sorenvind; our fix was to pin the version back to 0.17.10, so YMMV

@henrik242
Copy link

Oddly enough, @sorenvind; our fix was to pin the version back to 0.17.10, so YMMV

I've had both versions happening here for different developers. Fixing by downgrading to 0.17.10, and fixing by upgrading to 0.18.1.

@EugeneHlushko
Copy link

EugeneHlushko commented Jan 11, 2017

Happens to me on 0.18.1 yarn.

$ yarn --version
/usr/lib/node_modules/yarn/bin/yarn.js:62
      throw err;
      ^

Error: ENOENT: no such file or directory, open '/usr/lib/node_modules/yarn/node_modules/bl/node_modules/readable-stream/duplex.js'
    at Object.fs.openSync (fs.js:557:18)
    at Object.fs.readFileSync (fs.js:467:33)
    at Object.Module._extensions..js (module.js:581:20)
    at Module.load (module.js:490:32)
    at tryModuleLoad (module.js:449:12)
    at Function.Module._load (module.js:441:3)
    at Module.require (module.js:500:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/usr/lib/node_modules/yarn/node_modules/bl/bl.js:1:82)
    at Module._compile (module.js:573:32)

node: 0.7.1
npm: 3.10.9

steps to reproduce:
npm i -g yarn && yarn --version

@mweststrate
Copy link

Same here, had to downgrade

@cparjaszewski
Copy link

cparjaszewski commented Jan 16, 2017

works after downgrade from 0.18.x to 0.16.x

@chucknelson
Copy link

Not drudging this up, just linking it to the known Yarn issue: #5998

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
Labels
None yet
Projects
None yet
Development

No branches or pull requests