tools: remove disabling of already-disabled rule by Trott · Pull Request #6013

tools: remove disabling of already-disabled rule by Trott · Pull Request #6013 · nodejs/node

added buffer

Issues and PRs related to the buffer subsystem.

test

Issues and PRs related to the tests.

tools

Issues and PRs related to the tools directory.

labels

Apr 2, 2016

Trott changed the title ~~tools: remove require-buffer lint rule for tests~~ tools: enable require-buffer lint rule for tests

Apr 2, 2016

`require-buffer` is only enabled in the `lib` directory. There is no
need to disable it in `test`.

Trott changed the title ~~tools: enable require-buffer lint rule for tests~~ tools: remove disabling of already-disabled rule

Apr 2, 2016

jasnell pushed a commit that referenced this pull request

Apr 2, 2016

`require-buffer` is only enabled in the `lib` directory. There is no
need to disable it in `test`.

PR-URL: #6013
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Apr 5, 2016

`require-buffer` is only enabled in the `lib` directory. There is no
need to disable it in `test`.

PR-URL: #6013
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

This was referenced

Apr 5, 2016

MylesBorins pushed a commit that referenced this pull request

May 3, 2016

`require-buffer` is only enabled in the `lib` directory. There is no
need to disable it in `test`.

PR-URL: #6013
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

MylesBorins pushed a commit that referenced this pull request

May 6, 2016

`require-buffer` is only enabled in the `lib` directory. There is no
need to disable it in `test`.

PR-URL: #6013
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

MylesBorins pushed a commit that referenced this pull request

May 18, 2016

`require-buffer` is only enabled in the `lib` directory. There is no
need to disable it in `test`.

PR-URL: #6013
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>