Skip to content

Conversation

@mscdex
Copy link
Contributor

@mscdexmscdex commented Oct 1, 2016

Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)
  • cluster
Description of change

Removes an unused backlog argument.

CI: https://ci.nodejs.org/job/node-test-pull-request/4345/

EDIT: CI is green except for a flaky test.

@mscdexmscdex added the cluster Issues and PRs related to the cluster subsystem. label Oct 1, 2016
@fhinkel
Copy link
Member

Thanks! Landed in 9eb6179.

@fhinkelfhinkel closed this Oct 3, 2016
fhinkel pushed a commit that referenced this pull request Oct 3, 2016
PR-URL: #8877 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
@mscdexmscdex deleted the cluster-remove-unused-arg branch October 3, 2016 14:37
jasnell pushed a commit that referenced this pull request Oct 6, 2016
PR-URL: #8877 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Fishrock123 pushed a commit that referenced this pull request Oct 11, 2016
PR-URL: #8877 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clusterIssues and PRs related to the cluster subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@mscdex@fhinkel@bnoordhuis@santigimeno@cjihrig@MylesBorins