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.

Sign up for GitHub

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

Jump to bottom

lib,fs: use process.emitWarning instead of internal print dep warning #8166

Closed
wants to merge 4 commits into from
Closed

lib,fs: use process.emitWarning instead of internal print dep warning #8166

wants to merge 4 commits into from

Commits on Sep 2, 2016

  1. process: fix handling of process.noDeprecation in emitWarning

    @jasnell
    jasnell committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    46a69b4 View commit details
    Browse the repository at this point in the history

  2. lib: use emitWarning instead of printDeprecationMessage 

    The process.emitWarning() API should be used for printing
deprecation warning messages rather than directly using the
internal/util#printDeprecationMessage
    @jasnell
    jasnell committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    5aa3ea2 View commit details
    Browse the repository at this point in the history

  3. fs: use process.emitWarning to print deprecation warning 

    As an alternative to nodejs#6413, use
process.emitWarning() instead of the internal printDeprecationMessage
in order to avoid use of an internal only API.
    @jasnell
    jasnell committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    09d97e9 View commit details
    Browse the repository at this point in the history

  4. internal/util: remove printDeprecationWarning 

    Removes the internal/util printDeprecationWarning method
    @jasnell
    jasnell committed Sep 2, 2016
    Configuration menu
    Copy the full SHA
    2ef2c7f View commit details
    Browse the repository at this point in the history