First go at moving to xo

[OmgImAlexis]

@p0psicles @Thraxis

This uses https://github.com/sindresorhus/xo and https://github.com/sindresorhus/atom-linter-xo

• PR is based on the DEVELOP branch
• Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review
• Read contribution guide

[coveralls]

Coverage remained the same at 28.881% when pulling 76368186ad8f081cc865d6b2f325f913a1183e58 on jshint_to_xo into f90f3d3 on develop.

[coveralls]

Coverage decreased (-0.007%) to 28.874% when pulling 09205b3ee88253980af3bc7234bf06527b1ee498 on jshint_to_xo into f90f3d3 on develop.

[coveralls]

Coverage remained the same at 28.881% when pulling 09205b3ee88253980af3bc7234bf06527b1ee498 on jshint_to_xo into f90f3d3 on develop.

[coveralls]

Coverage decreased (-0.007%) to 28.874% when pulling 09205b3ee88253980af3bc7234bf06527b1ee498 on jshint_to_xo into f90f3d3 on develop.

[OmgImAlexis]

All of these can now be changed to use the jQuery function I added.

 if (!emby.host) {
     $('#emby_host').addClass('warning');
 } else {       
    $('#emby_host').removeClass('warning');     
 }

$('#emby_host').addRemoveWarningClass(emby.host);

[medariox]

So I have been testing this for a few hours, trying out everything I could etc. My JS knowledge is fairly limited, but everything seems to be running just fine so far.

What I noticed:

• My linter tells me that https://github.com/pymedusa/SickRage/blob/7dba22759fd4baaf30c13cabbc28fcf28a33f263/gui/slick/js/core.js#L3433 was used before it was defined. Isn't this an issue?
• There are some variables that are not camel case, they will need // eslint-disable-line camelcase to not fail the tests.

[OmgImAlexis]

The issue about the wizard is fine since it's defined in the vender.min.js file.

It's already passed travis so that's fine for now. The next PR I'm pushing has most of the other fixes but I didn't want to add onto an already extremely large PR.

[medariox]

Approved

Sounds good!

[fernandog]

@OmgImAlexis concluded?