Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@evmar
Copy link
Contributor

@evmar evmar commented Nov 2, 2017

  • Move setup instructions lower (most users will just be installing
    this from npm, so they don't need to know about bazel).
  • Document current workaround for setup.

See bazel-contrib/rules_nodejs#46 .

- Move setup instructions lower (most users will just be installing
  this from npm, so they don't need to know about bazel).
- Document current workaround for setup.

See bazel-contrib/rules_nodejs#46 .
@alexeagle
Copy link
Contributor

Give me a few minutes, I am hoping to just fix that bug rather than need to have the workaround...

@alexeagle
Copy link
Contributor

how about #642

@alexeagle
Copy link
Contributor

this commit merged in #642

@alexeagle alexeagle closed this Nov 2, 2017
@evmar evmar deleted the readme-yarn branch June 14, 2018 20:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants