73rhodes / Sublime-JSLint

JSLint for Sublime Text 2 and 3.
80 stars 14 forks source link

JSLint settings doesn't work on ST3 #29

Closed duranmla closed 10 years ago

duranmla commented 10 years ago

My problem is that I evn when on my configuration you will find:

    // if true, run jslint on save.
    ,"run_on_save" : false

it doesn't work and besides I can't add flag like browser to avoid errors like: 'window is not defined'

darrenderidder commented 10 years ago

Hi, what operating system and node versions are you running?

duranmla commented 10 years ago

I'm on Ubuntu 13.10

darrenderidder commented 10 years ago

Thanks, going to try and reproduce this on Ubuntu...

duranmla commented 10 years ago

Thanks that would be great

darrenderidder commented 10 years ago

Not able to reproduce on ubuntu 14.04 LTS and Sublime Text 3 build 3047. Could you post the contents of your user settings?

darrenderidder commented 10 years ago

Also can you please update via tools > package control > update packages > JSLint

duranmla commented 10 years ago

So should I update my OS necessarily?

my user setting for JSLint are the same as the default one.

https://gist.github.com/duranmla/1c90687c716eceec812f

I don't have the option to perform "Also can you please update via tools > package control > update packages > JSLint" if I go to tool menu context I don't have that option. Let me search how to update my packages.

duranmla commented 10 years ago

I make the upgrade by: Package Control -> Ctrl + Shift + P and then upgrade package and JSLint comes out, hit enter and the console shows a successfully process of upgrading.. However it doesn't solve the issue

darrenderidder commented 10 years ago

It shouldn't matter at all about 13.10 vs 14.04... could you post the entire contents of your user setting file, though?

duranmla commented 10 years ago

Yes of course, o give you a gist link with all my configuration do I make a mistake on it?

On Monday, June 16, 2014, Darren DeRidder notifications@github.com wrote:

It shouldn't matter at all about 13.10 vs 14.04... could you post the entire contents of your user setting file, though?

— Reply to this email directly or view it on GitHub https://github.com/darrenderidder/Sublime-JSLint/issues/29#issuecomment-46253695 .

Alexis Durán

duranmla commented 10 years ago

This is the link: https://gist.github.com/duranmla/1c90687c716eceec812f it's ok?

On Monday, June 16, 2014, Alexis Duran alexis.duranm@gmail.com wrote:

Yes of course, o give you a gist link with all my configuration do I make a mistake on it?

On Monday, June 16, 2014, Darren DeRidder <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

It shouldn't matter at all about 13.10 vs 14.04... could you post the entire contents of your user setting file, though?

— Reply to this email directly or view it on GitHub https://github.com/darrenderidder/Sublime-JSLint/issues/29#issuecomment-46253695 .

Alexis Durán

Alexis Durán

darrenderidder commented 10 years ago

Thanks. It's puzzling. Have you edited the user preferences by choosing menu Preference > Package Control > JSLint > user settings (or something like that... it varies by OS)?

duranmla commented 10 years ago

Yup actually If I don't remember wrong I read that it's recommended to copy a paste the default setting into the user settings to edit it as we want. So that is what I did, however my main concern is that I have to hit CTRL + B to run JSLint.

Of course I would like to add others preferences but that one is vital.

On Monday, June 16, 2014, Darren DeRidder notifications@github.com wrote:

Thanks. It's puzzling. Have you edited the user preferences by choosing menu Preference > Package Control > JSLint > user settings (or something like that... it varies by OS)?

— Reply to this email directly or view it on GitHub https://github.com/darrenderidder/Sublime-JSLint/issues/29#issuecomment-46255413 .

Alexis Durán

duranmla commented 10 years ago

I've just define puzzling hehe.. So I may upload my file or an screenshot when I get home cause now I don't have my laptop with me

On Monday, June 16, 2014, Alexis Duran alexis.duranm@gmail.com wrote:

Yup actually If I don't remember wrong I read that it's recommended to copy a paste the default setting into the user settings to edit it as we want. So that is what I did, however my main concern is that I have to hit CTRL + B to run JSLint.

Of course I would like to add others preferences but that one is vital.

On Monday, June 16, 2014, Darren DeRidder <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

Thanks. It's puzzling. Have you edited the user preferences by choosing menu Preference > Package Control > JSLint > user settings (or something like that... it varies by OS)?

— Reply to this email directly or view it on GitHub https://github.com/darrenderidder/Sublime-JSLint/issues/29#issuecomment-46255413 .

Alexis Durán

Alexis Durán

darrenderidder commented 10 years ago

CTRL-B would be probably JSHint... the shortcut for JSLint is CTRL-L by default. Is it possible you have edited the settings of JSHint instead of JSLint?

duranmla commented 10 years ago

Mmm weird I don't think so cause I leave jshint for JSLint and I'm pretty sure that I rid the JSLint setting however I'm going to double check to make sure and I'll let you now.

On Monday, June 16, 2014, Darren DeRidder notifications@github.com wrote:

CTRL-B would be probably JSHint... the shortcut for JSLint is CTRL-L by default. Is it possible you have edited the settings of JSHint instead of JSLint?

— Reply to this email directly or view it on GitHub https://github.com/darrenderidder/Sublime-JSLint/issues/29#issuecomment-46257635 .

Alexis Durán

darrenderidder commented 10 years ago

Any update?

duranmla commented 10 years ago

Excuse me, I've been very busy with a project please could you give me a few hours and I'll give you an update?. Thanks for answer and stay tuned I really need to solve this issue with my JSLint.

On Thursday, June 19, 2014, Darren DeRidder notifications@github.com wrote:

Any update?

— Reply to this email directly or view it on GitHub https://github.com/darrenderidder/Sublime-JSLint/issues/29#issuecomment-46632095 .

Alexis Durán

duranmla commented 10 years ago

Hi darrenderidder,

I'm quite happy, seems like after the update I should restart my computer or maybe just the ST, I'm not sure, but today when I wake up I review everything again and change my settings and it works.

However Ctrl + B give me the same output of JSLint and I don't have jshint (I had once but I don't have it now..)

Now I'm going to configure other stuff like detect 'window' I'm going to search for it.

Sorry if I make you lost your time, but I think that your first suggestion lead me to the solution but with all the work I may be was a little distracted, sorry again and thanks too much!

Warm regards

darrenderidder commented 10 years ago

yeah I forgot to say restart ST.