Skip to content

lib: remove unused variables - #1290

Closed
mscdex wants to merge 1 commit into
nodejs:v1.xfrom
mscdex:lint
Closed

mscdex wants to merge 1 commit into
nodejs:v1.xfrom
mscdex:lint

Conversation

@mscdex

@mscdex mscdex commented Mar 28, 2015

Copy link
Copy Markdown
Contributor

No description provided.

@bnoordhuis

Copy link
Copy Markdown
Member

LGTM but I'm unclear on whether changes to streams code need to go through https://github.com/iojs/readable-stream now. /cc @chrisdickinson

@bnoordhuis

Copy link
Copy Markdown
Member

@mscdex

mscdex commented Mar 29, 2015

Copy link
Copy Markdown
Contributor Author

The last I heard they are still pulling from this repo.

mscdex added a commit that referenced this pull request Mar 31, 2015
PR-URL: #1290
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@mscdex

mscdex commented Mar 31, 2015

Copy link
Copy Markdown
Contributor Author

Landed in 4dc6ae2.

@mscdex mscdex closed this Mar 31, 2015
@mscdex
mscdex deleted the lint branch March 31, 2015 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants