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

6242 backport #6869

Closed
wants to merge 2 commits into from
Closed

6242 backport #6869

wants to merge 2 commits into from

Conversation

Trott
Copy link
Member

@Trott Trott commented May 18, 2016

backport of #6242 for v4.x

/cc @thealphanerd

@nodejs-github-bot nodejs-github-bot added child_process Issues and PRs related to the child_process subsystem. repl Issues and PRs related to the REPL subsystem. util Issues and PRs related to the built-in util module. zlib Issues and PRs related to the zlib subsystem. labels May 18, 2016
@mscdex mscdex added the v4.x label May 19, 2016
Trott added 2 commits May 24, 2016 14:01
Correct alignment on variable assignments that span multiple lines in
preparation for lint rule to enforce such alignment.
Enforce alignment/indentation on variable assignments that span multiple
lines.
@MylesBorins
Copy link
Contributor

MylesBorins commented May 24, 2016

LGTM

landed in e2ccf62...146364b

MylesBorins pushed a commit that referenced this pull request May 24, 2016
Correct alignment on variable assignments that span multiple lines in
preparation for lint rule to enforce such alignment.

PR-URL: #6869
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
MylesBorins pushed a commit that referenced this pull request May 24, 2016
Enforce alignment/indentation on variable assignments that span multiple
lines.

PR-URL: #6869
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jun 23, 2016
Correct alignment on variable assignments that span multiple lines in
preparation for lint rule to enforce such alignment.

PR-URL: #6869
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jun 23, 2016
Enforce alignment/indentation on variable assignments that span multiple
lines.

PR-URL: #6869
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
Correct alignment on variable assignments that span multiple lines in
preparation for lint rule to enforce such alignment.

PR-URL: #6869
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jun 24, 2016
Enforce alignment/indentation on variable assignments that span multiple
lines.

PR-URL: #6869
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Johan Bergström <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Jun 24, 2016
@MylesBorins MylesBorins removed their assignment Dec 27, 2016
@Trott Trott deleted the 6242-backport branch January 13, 2022 22:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
child_process Issues and PRs related to the child_process subsystem. repl Issues and PRs related to the REPL subsystem. util Issues and PRs related to the built-in util module. zlib Issues and PRs related to the zlib subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants