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

Update index.js #28

Closed
wants to merge 1 commit into from
Closed

Update index.js #28

wants to merge 1 commit into from

Conversation

fridolinvii
Copy link

Change tty to tty-browserify. This resolved an issue i had during compiling.

Change tty to tty-browserify. This resolved an issue i had during compiling.
@knownasilya
Copy link
Owner

What was your issue and which version of node and npm did you use?

@knownasilya
Copy link
Owner

Comment from the tty-browserify repo: Browserify uses this module when you do require('tty') in a bundle. You should normally not use it directly.

Seems like you tried to use this module from the browser? tty is built into node and if you need to use this module in the browser you should shim tty with tty-browserify, but that should happen at the bundling/build tool level.

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.

None yet

2 participants