Skip to content

Downgrade Travis CI Node.js version to 14.9.0#1454

Merged
sindresorhus merged 2 commits into
sindresorhus:masterfrom
Giotino:ci-node-downgrade
Sep 10, 2020
Merged

Downgrade Travis CI Node.js version to 14.9.0#1454
sindresorhus merged 2 commits into
sindresorhus:masterfrom
Giotino:ci-node-downgrade

Conversation

@Giotino

@Giotino Giotino commented Sep 10, 2020

Copy link
Copy Markdown
Contributor

In order to keep using Travis CI correctly v14.9.0 has been specified instead of v14 (latest available).

This is a temporary measure, a rollback of the commit "stream: simpler and faster Readable async iterator" which broke nearly everything in Got has been planned for the next Node.JS release (v14.10.1) nodejs/node#35137

I don't know if downgrading Node.JS for CI is the right thing to do, but I think it won't do any harm while waiting for some fix for v14.10.0.

@sindresorhus @szmarczak

@sindresorhus sindresorhus changed the title Travis CI Node.JS 14 downgrade (to v14.9.0) Downgrade Travis CI Node.js version to 14.9.0 Sep 10, 2020
@sindresorhus sindresorhus merged commit 27470b5 into sindresorhus:master Sep 10, 2020
@Giotino Giotino deleted the ci-node-downgrade branch September 11, 2020 11:57
@Giotino

Giotino commented Sep 11, 2020

Copy link
Copy Markdown
Contributor Author

Now that the commit "stream: simpler and faster Readable async iterator" has been reverted this PR should also be reverted.

@sindresorhus

@sindresorhus

Copy link
Copy Markdown
Owner

@Giotino We have to wait until it's released and available on Travis.

@Giotino

Giotino commented Sep 11, 2020

Copy link
Copy Markdown
Contributor Author

@Giotino We have to wait until it's released and available on Travis.

As you can see here https://travis-ci.com/github/sindresorhus/got/jobs/383613035
NVM on Travis CI is installing v14.10.1 for the 14 target.

sindresorhus added a commit that referenced this pull request Sep 11, 2020
@sindresorhus

Copy link
Copy Markdown
Owner

0e0eefd

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.

2 participants