Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

require-buffer lint fixer #17144

Closed
wants to merge 5 commits into from
Closed

Commits on Nov 20, 2017

  1. Fixes: nodejs#16636

    Adds `require-buffer` lint fixer.
    If the buffer module is not required while `Buffer` is used, require the `buffer` module.
    If the file has a `'use strict';` line, add the require after it on a separate line.
    If the file does not have any (currently impossible with the `strict` rule) add it after the first comment (before the real code starts).
    Bamieh committed Nov 20, 2017
    Configuration menu
    Copy the full SHA
    1393def View commit details
    Browse the repository at this point in the history
  2. add require-buffer fixer tests

    Bamieh committed Nov 20, 2017
    Configuration menu
    Copy the full SHA
    04bcdd3 View commit details
    Browse the repository at this point in the history
  3. add final test scenario

    Bamieh committed Nov 20, 2017
    Configuration menu
    Copy the full SHA
    694a8c9 View commit details
    Browse the repository at this point in the history

Commits on Nov 23, 2017

  1. Merge branch 'master' of https:/nodejs/node into lint-fix…

    …er-require-buffer
    Bamieh committed Nov 23, 2017
    Configuration menu
    Copy the full SHA
    e6d915e View commit details
    Browse the repository at this point in the history

Commits on Nov 25, 2017

  1. Configuration menu
    Copy the full SHA
    cbe7123 View commit details
    Browse the repository at this point in the history