Skip to content

Don't do console.log.apply(this, ...).#274

Merged
nfischer merged 1 commit intoshelljs:masterfrom
ariporad:fix/255
Jan 15, 2016
Merged

Don't do console.log.apply(this, ...).#274
nfischer merged 1 commit intoshelljs:masterfrom
ariporad:fix/255

Conversation

@ariporad
Copy link
Copy Markdown
Contributor

@ariporad ariporad commented Jan 8, 2016

Fixes #255.

@ariporad ariporad added the fix Bug/defect, or a fix for such a problem label Jan 9, 2016
@ariporad ariporad self-assigned this Jan 9, 2016
@nfischer
Copy link
Copy Markdown
Member

Could we re-ping this to trigger the CI?

@nfischer
Copy link
Copy Markdown
Member

This looks good, but I'd like to verify that it passes the CI before merging. Perhaps we can rebase off master once #267 is merged (since the log() function uses console.error in that PR)?

Aside from that, I expect this to work on both Windows and *nix.

@nfischer
Copy link
Copy Markdown
Member

@ariporad if the merge conflicts can be resolved, I can merge this 👍

@ariporad
Copy link
Copy Markdown
Contributor Author

@nfisher: there are none. Merging

EDIT: never mind, caching. I will fix when I'm at a computer

@nfischer
Copy link
Copy Markdown
Member

I think the only issue is that I changed a console.log to a console.error. Should be an easy fix.

@ariporad
Copy link
Copy Markdown
Contributor Author

@nfischer: Fixed. git refork to the rescue!

@nfischer
Copy link
Copy Markdown
Member

@ariporad Awesome! I'll merge.

nfischer added a commit that referenced this pull request Jan 15, 2016
Don't do `console.log.apply(this, ...)`.
@nfischer nfischer merged commit 309784a into shelljs:master Jan 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug/defect, or a fix for such a problem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants