Warn when using the Buffer constructor

kgryte · kgryte · commit f36494954a00 · 2017-06-23T13:57:50.000-07:00
diff --git a/etc/eslint/rules/nodejs.js b/etc/eslint/rules/nodejs.js
@@ -87,6 +87,25 @@ rules[ 'global-require' ] = 'off';
 */
 rules[ 'handle-callback-err' ] = [ 'error', '^(err|error)$' ];
 
+/**
+* Warn when using the `Buffer` constructor.
+*
+* @name no-buffer-constructor
+* @memberof rules
+* @type {string}
+* @default 'warn'
+* @see [no-buffer-constructor]{@link http://eslint.org/docs/rules/no-buffer-constructor}
+*
+* @example
+* // Bad...
+* var buf = new Buffer( [ 1, 2, 3 ] );
+*
+* @example
+* // Good...
+* var buf = Buffer.from( [ 1, 2, 3 ] );
+*/
+rules[ 'no-buffer-constructor' ] = 'warn'; // TODO: revisit once all Buffer usage includes support for older Node versions
+
 /**
 * Only allow `require`d modules to be grouped together.
 *
