-
Notifications
You must be signed in to change notification settings - Fork 6.3k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
fix changelog links for new 0.12 and 0.10 releases #505
Conversation
r=@nodejs/website |
541e6f8
to
9d3a781
Compare
LGTM |
@@ -9,5 +9,10 @@ module.exports = function (version) { | |||
return `https://github.com/nodejs/node/blob/${version}/CHANGELOG.md` | |||
} | |||
|
|||
// 0.12.8+ and 0.10.41+ releases come from the new repo | |||
if (semver.satisfies(version, '~0.12.8 || ~0.10.41')) { | |||
return `https://github.com/nodejs/node/blob/${version}/ChangeLog` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does this not need to be ChangeLog.md?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thealphanerd It was not markdown until io.js v1.0.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh my bad... I was misunderstanding what the bug was. nothing to see here :D
Ignore the noise above... LGTM |
fix changelog links for new 0.12 and 0.10 releases
Fixes: #504
PR-URL: #505