Mention yarn in the README.md #345
Conversation
|
Looks good. Maybe skip "pre-compiled", i doesn't make much sense for npm and yarn. |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Starefossen
force-pushed
the
docs/mention-yarn
branch
2 times, most recently
from
df184ed to
c0e7516
Compare
|
Updated 😄 |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
|
It seems a few builds a failing because of trying to install yarn from npm. Do yo guys think we should open a meta issues with the error that occurs in the title for a few weeks to try and catch people having issues with this? @nodejs/docker |
|
I think that would be a good idea. Maybe also mention in the issue why we aren't installing yarn via npm (space efficiency) |
|
Great suggestion @LaurentGoderre. Also @pesho ++ feel free to just fix the spelling directly to the branch, I will be away from the terminal for the rest of the day. |
Starefossen
force-pushed
the
docs/mention-yarn
branch
from
c0e7516 to
efbb5f5
Compare
Starefossen
force-pushed
the
docs/mention-yarn
branch
from
efbb5f5 to
d46f9eb
Compare
|
Fixed up last nitpick. Merging 😄 |
I think we should mention what is installed in the Image. Should we add some links or write some more about the different parts?