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

More instructions in readme? #34

Closed
evancz opened this issue Feb 23, 2016 · 4 comments
Closed

More instructions in readme? #34

evancz opened this issue Feb 23, 2016 · 4 comments

Comments

@evancz
Copy link

evancz commented Feb 23, 2016

I sat with @christinecha as she got Elm installed and started playing around with it for the first time. It was weirdly difficult to get syntax highlighting working for Elm in Atom though.

  1. She installed via npm but the highlighting did not work.
  2. She googled around to figure out where the heck apm is in the editor itself.
  3. When she finally found it, it said that language-elm was already installed.
  4. She uninstalled, and reinstalled through the UI.

Then it worked. I don't really know why it was that way, but I feel like having more comprehensive instructions in the README could save people a bunch of time. Maybe just saying how to get to the package search thing in the editor? Not really sure!

@brianegan
Copy link
Collaborator

Ouch. Not a good first time experience. Thanks for the feedback!

Unfortunately, without more context, I couldn't determine why the command line installation failed, but hopefully I can fix our documentation.

I'll write up some instructions for the README :)

brianegan pushed a commit that referenced this issue Feb 29, 2016
brianegan pushed a commit that referenced this issue Feb 29, 2016
brianegan pushed a commit that referenced this issue Feb 29, 2016
@brianegan
Copy link
Collaborator

@evancz Hey there, I put together some instructions! When you get a chance, could you please give them a gander and let me know if they help alleviate the issue you are describing?

Link to updated Readme:
https://github.com/edubkendo/atom-elm/blob/better-instructions/README.md

edubkendo added a commit that referenced this issue Feb 29, 2016
Fixes #34: Add installation instructions
@christinecha
Copy link

Looks good to me! :)

Did we figure out why it wasn't fully installing when I ran "apm install
language-elm", though?

On Mon, Feb 29, 2016 at 6:10 PM, Brian Egan notifications@github.com
wrote:

@evancz https://github.com/evancz Hey there, I put together some
instructions! When you get a chance, could you please give them a gander
and let me know if they solve the issue you are describing?

Link to updated Readme:
https://github.com/edubkendo/atom-elm/blob/better-instructions/README.md


Reply to this email directly or view it on GitHub
#34 (comment).

Christine Cha

DEVELOPER + DESIGNER // CHRISTINECHA.COM http://christinecha.com
NYC LEAN IN CHAPTER LEADER // NYCLEANIN.COM http://nycleanin.com

NEW PROJECTS --
*CIRCLEMATCH http://christinecha.github.io/circlematch/: *a minimalistic
twist on the classic 8-piece slide puzzle.

@brianegan
Copy link
Collaborator

@christinecha Yay! Glad it fixed part of the problem. And sorry, totally my bad, should have cc'd you as well on that comment as well. With regards to apm failing, I'd have to say: I really have no idea :(

Would you be willing to try installing it again via apm? Does it fail every time for you? Did it give any additional feedback, or just say Installing language-elm to... (check mark)?

My general debugging strategy in this case would be to 1. reproduce the error, or 2. get some detailed logs from apm and look through them to see where the installation failed. Unfortunately, I haven't been able to reproduce the error, and I don't see a way to get any logs out of apm. Therefore, I don't feel I have the tools to say what could have gone wrong. If you can reproduce consistently and / or provide logs, I'd be happy to dig in further!

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

No branches or pull requests

3 participants