Skip to content

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: #13498

PR-URL: #19216
Refs: #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 authored and MylesBorins committed Mar 20, 2018
1 parent a4a4819 commit 411f3e0
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 411f3e0

Please sign in to comment.