Skip to content
This repository was archived by the owner on Dec 15, 2018. It is now read-only.

Convert node versions return by Travis API to Strings#12

Merged
pvdlg merged 1 commit intomasterfrom
fix-build-leader
Sep 21, 2017
Merged

Convert node versions return by Travis API to Strings#12
pvdlg merged 1 commit intomasterfrom
fix-build-leader

Conversation

@pvdlg
Copy link
Member

@pvdlg pvdlg commented Sep 21, 2017

Travis API return the Node version as a String or a Number based on the
way it's defined in .travis.yml (node_js: '8' => String / node_js: 8 =>
Number).
semver expect a String and consider invalid the versions expressed as
a Number

Fixes semantic-release/semantic-release#347

Travis API return the Node version as a String or a Number based on the
way it's defined in .travis.yml (node_js: '8' => String / node_js: 8 =>
Number).
`semver` expect a String and consider invalid the versions expressed as
a Number

Fix #347
Copy link
Member

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

great catch 👍

@pvdlg pvdlg merged commit 4762a06 into master Sep 21, 2017
@pvdlg pvdlg deleted the fix-build-leader branch September 21, 2017 02:09
geraintwhite added a commit to itsapi/github-listener that referenced this pull request Sep 21, 2017
hack is no longer necessary as bug is fixed by [this](semantic-release/travis-deploy-once#12)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants