Нужен ли JSHint разработчикам использующие IDE WebStorm?

Question

[Иван Антонов]

Есть ли смысл настраивать JSHint в gulp, если в WebStorm'е и так есть анализ javascript?

Answer 1

[Денис Инешин]

Если вопрос заключается в том, что имеет ли в смысл встраивать одинаковые проверки и веб-шторм и в сборщик, то да, имеет и вот почему:
1. Если вы работаете один, то всегда возможна ситуация, что вы подправите код не со своего рабочего места, где стоит настроенный веб шторм, а хз откуда.
2. Если вы работаете в команде, то вопрос вообще не обсуждается даже, настроить каждому IDE, или настроить один раз сборщик, общий для всех?

Comments

[Иван Антонов]

а если я настрою минификацию JS при сборки проекта?

[Денис Инешин]

Иван Антонов: причем тут минификация? Она происходит уже после того, как исходный код проверен на правильность.

[Иван Антонов]

Денис Инешин: а ведь точно :)

Answer 2

[Константин Китманов]

Там разные проверки — некоторые пересекаются, некоторые нет. Не помешает иметь обе, одна голова хорошо, а две лучше.

WebStorm, кстати, поддерживает JSHint, JSLint и ESLint. Последний — самый гибкий и непрерывно развивающийся, я бы порекомендовал именно его.

Comments

[Иван Антонов]

обновил вопрос :) из-за недостатка знаний упустил этот момент.

[Константин Китманов]

Иван Антонов: имхо, имеет. Я в некоторых проектах делаю сборку скриптов только если они пройдут jscs, eslint и юнит-тесты. Зачем собирать, если скрипт заведомо не рабочий?:)
Ну и аргумент про дополнительную проверку все еще в силе.

А вот почему jshint не дает работать BrowserSync'у — это отдельный вопрос:)

[Иван Антонов]

Константин Китманов: я сейчас потестил и понял, что дело даже не в BrowserSync'у
https://github.com/antonowano/html5-boilerplate/bl...
если js-файл не пуст, то при сборке проекта - хинт возвращает ошибку даже на console.log

[Иван Антонов]

Константин Китманов:
events.js:72
throw er; // Unhandled 'error' event
^
Error: ←[1mMissing line feed at file end←[22m at ←[32mmain.js←[39m :
←[90m 1 |←[39mconsole.log('File `main.js` upload!');
←[90m----------------------------------------------^←[39m

[Константин Китманов]

Иван Антонов: вариантов решения миллион и маленькая тележка.
* Задизейблить jshint для этой строчки, типа такого: stackoverflow.com/questions/12977661/is- there-a-way-to-suppress-jshint-warning-for-one-given-line
* Закомментить или убрать эту строчку из исходников
* Разрешить console.log в настройках jshint

[copal]

Константин Китманов: eslint.org/demo если пройти по ссылке и проверить код ниже, то в нем три ошибки. Но разве это так?

var Test = (function(){
	function Test(){}

	Test.prototype.say = function () {
		console.log('Hello!');
	};

	return Test;
})();

[Константин Китманов]

copal: вообще-то да:) Но, по хорошему, надо настраивать под себя, eslint это позволяет.

[Иван Антонов]

Константин Китманов: ппц однако. пишу простую функцию уже целый час
events.js:72
throw er; // Unhandled 'error' event
^
Error: ←[1mInvalid line break←[22m at ←[32mmain.js←[39m :
←[90m 1 |←[39mfunction main() {
←[90m-------------------------^←[39m
←[90m 2 |←[39m return 'ogogo';
←[90m 3 |←[39m}

что за ошибка, этот хинт больше бесит чем помогает.

[Иван Антонов]

function main() {
    return 'ogogo';
}

main();

[Константин Китманов]

Иван Антонов: вы думаете, я все ошибки jshint'а наизусть помню? у меня есть занятия поинтереснее, чем гуглить тексты ошибок. Можно предположить, что стоит виндовый перенос строки (или линуксовый, или маковский).

[Иван Антонов]

Константин Китманов: спасибо, мне это помогло

[Константин Китманов]

Иван Антонов: я советую все-таки взять eslint, и настроить его под себя — чтобы представлять, какие опции в нем есть, на что он проверяет и что считает ошибкой.