Skip to content

Conversation

@tingwai
Copy link

@tingwai tingwai commented Jul 12, 2019

Based on the current .travis.yml, node version 10, 11, 12 are supported. I have updated the package.json to reflect that. Otherwise amqplib fails to install when users have flag engine-strict=true in their .npmrc.

package.json Outdated
},
"engines": {
"node": ">=0.8 <=9"
"node": ">=0.8 <=11"
Copy link

Choose a reason for hiding this comment

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

I guess it should be <= 12

Copy link
Author

Choose a reason for hiding this comment

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

Ah you're right, updating now

@squaremo
Copy link
Collaborator

You'll need to rebase on master, which has a fix for the CI build (see #526).

@tingwai
Copy link
Author

tingwai commented Jul 15, 2019

You'll need to rebase on master, which has a fix for the CI build (see #526).

Thanks for the heads up. PR is rebased now

@squaremo squaremo merged commit 7adaea0 into amqp-node:master Jul 16, 2019
@squaremo
Copy link
Collaborator

Thanks @tingwai (and @L2jLiga for the assist)!

@tingwai
Copy link
Author

tingwai commented Jul 16, 2019

@squaremo Thanks for the merge. Can you release a new version to npm? This will help resolve the engine strict issues in my CI

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.

3 participants