labels: map dont-land-on-v4(6).x for http2 (nodejs#152)

MylesBorins · phillipj · commit 964ee98d3f64 · 2017-10-08T00:59:42.000+02:00
* labels: map dont-land-on-v4(6).x for http2

* labels: added tests for map dont-land-on-v4(6).x for http2

* labels: map dont-land-on-v4(6).x for lib/http2
diff --git a/lib/node-labels.js b/lib/node-labels.js
@@ -30,7 +30,7 @@ const subSystemLabelsMap = new Map([
   [/^src\/node_zlib/, ['c++', 'zlib']],
   [/^src\/tracing/, ['c++', 'tracing']],
   [/^src\/node_api/, ['c++', 'n-api', 'dont-land-on-v4.x', 'dont-land-on-v6.x']],
-  [/^src\/node_http2/, ['c++', 'http2']],
+  [/^src\/node_http2/, ['c++', 'http2', 'dont-land-on-v4.x', 'dont-land-on-v6.x']],
 
   // don't label python files as c++
   [/^src\/.+\.py$/, 'lib / src'],
@@ -73,8 +73,8 @@ const subSystemLabelsMap = new Map([
   [/^deps\/uv\//, 'libuv'],
   [/^deps\/v8\/tools\/gen-postmortem-metadata\.py/, ['V8 Engine', 'post-mortem']],
   [/^deps\/v8\//, 'V8 Engine'],
-  [/^deps\/nghttp2\/nghttp2\.gyp/, ['build', 'http2']],
-  [/^deps\/nghttp2\//, 'http2'],
+  [/^deps\/nghttp2\/nghttp2\.gyp/, ['build', 'http2', 'dont-land-on-v4.x', 'dont-land-on-v6.x']],
+  [/^deps\/nghttp2\//, ['http2', 'dont-land-on-v4.x', 'dont-land-on-v6.x']],
   [/^deps\/([^/]+)/, '$1'],
 
   /* JS subsystems */
@@ -87,6 +87,7 @@ const subSystemLabelsMap = new Map([
   [/^lib\/\w+\/v8_prof_/, 'tools'],
   [/^lib\/\w+\/socket_list/, 'net'],
   [/^lib\/\w+\/streams$/, 'stream'],
+  [/^lib\/.*http2/, ['http2', 'dont-land-on-v4.x', 'dont-land-on-v6.x']],
   [/^lib\/internal\/url\.js$/, ['url-whatwg',
                                 'dont-land-on-v4.x', 'dont-land-on-v6.x']],
   // All other lib/ files map directly
diff --git a/test/unit/node-labels.test.js b/test/unit/node-labels.test.js
@@ -615,16 +615,17 @@ tap.test('label: "build" when ./android-configure has been changed', (t) => {
 });
 
 [
-  [ ['http2'],
+  [ ['http2', 'dont-land-on-v4.x', 'dont-land-on-v6.x'],
     ['lib/http2.js',
      'lib/internal/http2/core.js',
      'deps/nghttp2/lib/nghttp2_buf.c'] ],
-  [ ['c++', 'http2'],
+  [ ['c++', 'http2', 'dont-land-on-v4.x', 'dont-land-on-v6.x'],
     ['src/node_http2.cc',
      'src/node_http2.h',
      'src/node_http2_core.h',
      'src/node_http2_core-inl.h'] ],
-  [ ['build', 'http2'], ['deps/nghttp2/nghttp2.gyp'] ],
+  [ ['build', 'http2', 'dont-land-on-v4.x', 'dont-land-on-v6.x'],
+    ['deps/nghttp2/nghttp2.gyp'] ],
   [ ['doc', 'http2'], ['doc/api/http2.md'] ]
 ].forEach((info) => {
   const labels = info[0]
