Skip to content
This repository has been archived by the owner on Oct 15, 2020. It is now read-only.

Commit

Permalink
test: remove flaky status for test-npm-install
Browse files Browse the repository at this point in the history
I haven't seen test-npm-install fail on arm in a long time. There is no
open issue for it being unreliable. Remove flaky designation in
parallel.status file. (If it turns out it is flaky, it's easy enough to
open an issue and restore the flaky designation.)

Ref: nodejs/node#13498

PR-URL: nodejs/node#19216
Refs: nodejs/node#13498
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
  • Loading branch information
Trott committed Mar 10, 2018
1 parent 0eec073 commit 9757906
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion test/parallel/parallel.status
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ test-postmortem-metadata: PASS,FLAKY
[$system==macos]

[$arch==arm || $arch==arm64]
test-npm-install: PASS,FLAKY

[$system==solaris] # Also applies to SmartOS

Expand Down

0 comments on commit 9757906

Please sign in to comment.