Skip to content
This repository has been archived by the owner on Feb 19, 2022. It is now read-only.

Update language for aliasing node_modules/.bin #38

Merged
merged 2 commits into from Mar 3, 2016
Merged

Conversation

zachhale
Copy link
Contributor

@zachhale zachhale commented Mar 1, 2016

To keep consistent with builder-react-app

To keep consistent with builder-react-app
`./node_modules/.bin` in `PATH` like:

```sh
export PATH="${PATH}:./node_modules/.bin"
$ export PATH="${PATH}:./node_modules/.bin"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably want to remove the $ since this is meant to be included in something like a .bashrc, not issued on the command line (unless you want to have that be an option?)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, good call!

@ryan-roemer
Copy link
Member

@zachhale -- One nit, after which rest LGTM

zachhale added a commit that referenced this pull request Mar 3, 2016
Update language for aliasing `node_modules/.bin`
@zachhale zachhale merged commit 22a0cbb into master Mar 3, 2016
@zachhale zachhale deleted the docs-path-update branch March 3, 2016 22:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants