Skip to content

Add "Troubleshooting" link to install guide #146

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

Closed
wants to merge 1 commit into from

Conversation

milesfrain
Copy link
Contributor

@hdgarrood
Copy link
Collaborator

I'd rather have the npm installer print this link if it fails. That way, if it does fail, the info you need is immediately right there; you don't have to go back to the purescript.org webpage to find it.

@milesfrain
Copy link
Contributor Author

If you point me to the section in the installer of where to add this, I'll attempt to make that change.

@hdgarrood
Copy link
Collaborator

I think here would be the best place for it: https://github.com/purescript/npm-installer/blob/9761cd2778853bd661761990297fc80fa6e3755b/index.js#L315-L344

I think it should be fine to unconditionally append a couple of newlines and then this link to erroredTask.message after that property has been set initially, but if you could try to test it with each of the branches of the if statement which starts as if (err.code === 'ERR_UNSUPPORTED_PLATFORM' ... then that would be ideal.

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