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

root node_modules folder #130

Merged
merged 1 commit into from
Sep 26, 2014
Merged

root node_modules folder #130

merged 1 commit into from
Sep 26, 2014

Conversation

paulvi
Copy link
Contributor

@paulvi paulvi commented Sep 26, 2014

node_modules folder here make this project look like Node.js project but it is not.
Tern is inside node_modules/tern

I would have other name, e.g. js or ternjs

Also node_modules/.bin is weird.

`node_modules` folder here make this project look like Node.js project but it is not.
Tern is inside node_modules/tern
@paulvi paulvi changed the title Create README.md root node_modules folder Sep 26, 2014
angelozerr added a commit that referenced this pull request Sep 26, 2014
root node_modules folder
@angelozerr angelozerr merged commit d219d12 into angelozerr:master Sep 26, 2014
@paulvi paulvi deleted the patch-1 branch September 28, 2014 06:40
@paulvi
Copy link
Contributor Author

paulvi commented Sep 28, 2014

@angelozerr adding file was just a way to point to exact location

I think core/tern.core/node_modules/tern can be maid one level down

This is not major, of course, but related to #35

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.

2 participants