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

doc: refactor function expressions to arrow functions #4832

Closed
wants to merge 1 commit into from
Closed

doc: refactor function expressions to arrow functions #4832

wants to merge 1 commit into from

Commits on Jan 25, 2016

  1. doc: replace function expressions with arrow functions 

    This commit replaces multiple usages of `function(){}` with ES2015
arrow functions in places it was forgotten earlier. The goal is to
make the docs more consistent since other functions were already
replaced with ES2015 arrows.

In addition, it fixes invalid syntax in modules.markdown to valid
syntax as well as remove `var self = this` pattern usages in the code
where they are now possible to avoid through arrow functions.

PR-URL: nodejs#4832
Reviewed-By: Roman Reiss <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
    @benjamingr
    benjamingr committed Jan 25, 2016
    Configuration menu
    Copy the full SHA
    aabde97 View commit details
    Browse the repository at this point in the history