Not only does it not have any regression tests, it also fails the CI check, and ... | Hacker News

Hacker Newsnew | past | comments | ask | show | jobs | submit

		lhorie on Feb 22, 2018 \| parent \| context \| favorite \| on: Do not use NPM 5.7 Not only does it not have any regression tests, it also fails the CI check, and it's already merged to the next branch. https://github.com/npm/npm/pull/19889 This kind of thing disintegrates my confidence on npm as a project.

TheDong on Feb 22, 2018 [–]

It, in fact, did pass the CI testing. The commit in question with the red X (7dff9d6) was pushed as a branch and then passed here [1].

After passing the test, the PR was made and merged, and the PR-test failed because it branch was already merged and travis-CI has races around that.

[1]: https://travis-ci.org/npm/npm/builds/344892198?utm_source=gi...

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact