Github user sardell commented on a diff in the pull request:
https://github.com/apache/metron/pull/1096#discussion_r213361764
--- Diff: metron-interface/metron-alerts/.nvmrc ---
@@ -0,0 +1 @@
+v9.11.1
--- End diff --
Thanks for pointing this out! The maven configuration is fine because we
specify the npm version to install in the pom.xml (6.2.0). However, this will
effect developers working on Metron. As far as I can tell, there isn't a way
for us to include an npm version in our .nvmrc, so I think the next best thing
we can do is add a step to the README that instructs developers to install npm
6.2.0.
---
