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 hint - errors when importing packages #2193

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benkoshy
Copy link

@benkoshy benkoshy commented Apr 12, 2021

Quick Summary:

  • I was trying to import a package and I received this message here:

image

  • What does this mean? The message is helpful but does not pose the solution required: how to import or install the library?
  • This PR goes a small way towards alleviating that problem. I feel that this is in keeping with the goals of the elm compiler: clear error messages, and clear solutions to the problems. For the advanced users, the solution may be obvious -- not so for others.
  • Note: this might not be the best solution. "You can install packages like this: elm install elm/http". we are hard coding the solution: elm install elm/http. perhaps the compiler should be smart enough to suggest the correct module to install?
  • As always, these PRs are merely ideas for consideration. If they do not align with the goals of the library etc. feel free to close without compunction.

Terms of contribution

  • I am in agreement with the terms of contribution - the form has been completed and sent to the email address.

Ben

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.

1 participant