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

incredibly slow with many files #70

Open
bounceme opened this issue Jun 21, 2016 · 13 comments
Open

incredibly slow with many files #70

bounceme opened this issue Jun 21, 2016 · 13 comments

Comments

@bounceme
Copy link

can the file globbing be constrained to only required files, or made to not recurse search through folders?

@bounceme
Copy link
Author

why not just use whatever is loaded by tern as that info is available
http://ternjs.net/doc/manual.html#req_files

@angelozerr
Copy link
Owner

How to use tern-lint? With an editor? With line command? tern-lint

In tern.java (Eclipse), it uses the first (validates a given file" and it uses tern script path to know if file must be validated or not. If you use tern.java, you must exclude node_modules folder with yoru JSDT Install Path.

@bounceme
Copy link
Author

i'm using the vim integration provided by syntastic vim-syntastic/syntastic@5462fdd
this was just made a few hours ago, would you be able to make any suggestions on that commit?

@bounceme
Copy link
Author

bounceme commented Jun 21, 2016

really, i think the easiest option is using an environment variable or arg to exclude folders, is this already implemented? or is there another way?

@angelozerr
Copy link
Owner

It seems that you are using tern-lint command https://github.com/angelozerr/tern-lint/blob/master/bin/lint is that?

If it that, try to create a .tern-project with loadEagerly like described in the tern manual.

@bounceme
Copy link
Author

why is loadEagerly needed? tern can resolve require statements and parse necessary files that way

@bounceme
Copy link
Author

could this be run on a file rather than a directory?

@bounceme
Copy link
Author

bounceme commented Jun 22, 2016

would you consider adding this to the default config, or something similar?

    "dontLoad": [
        "+(node_modules|bower_components|min)/**/*.js"
    ],

@angelozerr
Copy link
Owner

You have the same config than .tern-project. So you can use dontLoad too. I want to have the same behaviour than ternjs, so if ternjs doesn't add this default config, I don't want to add it.

@bounceme
Copy link
Author

then using tern for autocomplete wouldn't work, but anyways the method of using load eagerly seems less effective than terns ability to find dependencies

@angelozerr
Copy link
Owner

then using tern for autocomplete wouldn't work,

My idea is to have the same scope for automplete and lint.

@bounceme
Copy link
Author

hey, i'll probably create a p.r. touching this function, to allow for a filename to be used, which will be the only file initially loaded with the server,would you accept something like this?
https://github.com/angelozerr/tern-lint/blob/master/bin/lint#L128-L155

@angelozerr
Copy link
Owner

I have given you an answer. I think it's a bad idea to use tern-lint command. I think you should use "lint" query (as "completion" query).

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

2 participants