Skip to content

lib: remove unused variables#1290

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

lib: remove unused variables#1290
mscdex wants to merge 1 commit intonodejs:v1.xfrom
mscdex:lint

Conversation

@mscdex
Copy link
Copy Markdown
Contributor

@mscdex mscdex commented Mar 28, 2015

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
Copy link
Copy Markdown
Contributor Author

mscdex commented Mar 29, 2015

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
Copy link
Copy Markdown
Contributor Author

mscdex commented Mar 31, 2015

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