trailing comma да, тоже всегда отключаю и не понимаю зачем включили они.

ну у этого правила есть плюсы и огромный минус - ппц как непривычно

trailing comma — очень удобно. Довольно просто менять элементы в коллекциях местами

и удаление последнего никак не аффектит предыдущие

собственно из той же области, что и multivar

минус правда в том, что в очень старых ie будет синтаксическая ошибка

и удаление последнего никак не аффектит предыдущие

Так eslint подсветит или укажет что там ошибка, если не правильно. Единственное что, при комите в git не подсвечиваются такие затронутые строчки.

тут вопрос в дополнительной работе

придется вручную удалить запятную с предыдущей строки

т.е. с trailling comma меньше правок выходит.

Ну и в git чище коммиты с обязательной запятой, всегда ее ставлю.

jscs все сам за тебя делает (если вдруг забыл что).

airbnb codestyle использую и с большей частью правил согласен и доволен с другой стороны - некоторые вещи слишком строги, другие сугубо итог внутрикомандного соглашения и по сути опциональны постепенно многие такие правила отрубил или модифицировал как базу, этот кодстайл я бы рекомендовал всем более менее большим проектам а опциональные детали уже обрабатываются напильником на месте

Кстати, по guidestyle. 7.12 Never mutate parameters. eslint: no-param-reassign Зачем извращаться с вызовом hasOwnProperty через прототип Object'a?

Вызов obj.hasOwnProperty как-то может повлиять на что-то? Или это защита от undefined?

мб защита от случаев когда obj не объект?

защита от того что объект не наследует прототип объекта

@DenisIzmaylov зачем его переводить?

Наверное, это новость, но в России у большинства разработчиков проблемы с английским :)

Наверное, это новость, но в России у большинства разработчиков проблемы с английским :)

Думаю, проблема преувеличена

Ошибаешься :)

Хм, ни разу не встречал разработчика, который бы не говорил по-английски

Хотя бы не читал

Впрочем, это мой сугубо личный опыт, выборка нерепрезентативна

Все-таки корреляция между владением английским и компетентностью есть

мб защита от случаев когда obj не объект?

Может быть.

мне постоянно приходится тыкать пару человек в тиме за не правильно написанные по английски переменные и вот это вот все

Ну то есть я бы не взял разработчика, который не сможет на стековерфлоу ответ найти

защита от того что объект не наследует прототип объекта

Или кто-то перезаписал hasOwnProperty :)

да неправильное написани и говорение очень далеко от понимания технического английского на тему инструмента, которым пользуешься

Впрочем, это мой сугубо личный опыт, выборка нерепрезентативна

У тебя слишком много друзей из 1С

мне постоянно приходится тыкать пару человек в тиме за не правильно написанные по английски переменные и вот это вот все

Соболезную...

Или кто-то перезаписал hasOwnProperty :)

всякое может быть :)

У тебя слишком много друзей из 1С

Сложная шутка :с

всякое может быть :)

Реально первый раз такой подход встречаю. Но это же слишком длинно и как-то коряво выглядит. Неужто нет подхода попроще? :)

Особенно учитывая, что styleguide просто переполнен всякими штуками, которые нацелены на упрощение понимания кода.

Понимание и предсказуемость

Airbnb-codestyle хорош, я его даже в Redux впилил прошлым летом :) Но сейчас я настолько влюбился в стиль no-semicolons, что хоть отдельный CS создавай

Денис, предсказуемость поведения, наверное да, но встретив такую конструкцию const key = Object.prototype.hasOwnProperty.call(obj, 'key') ? obj.key : 1; с ходу сложно конкретно понять, зачем так извращаться.

@DenisIzmaylov eslint-preset-standard ?

Денис, предсказуемость поведения, наверное да, но встретив такую конструкцию const key = Object.prototype.hasOwnProperty.call(obj, 'key') ? obj.key : 1; с ходу сложно конкретно понять, зачем так извращаться.

Вот это перебор :) Где там такое советуют?

const hasOwnProperty = (obj, key) => {}.hasOwnProperty.call(obj, key);

https://github.com/airbnb/javascript#functions--mutate-params

Там может быть проверка в for-each?

Нет там такой проверки.

Уже сделали предположение, что hasOwnProperty может быть перезаписано или существует предположение о том, что в качестве аргумента могут передать не только лишь объект...

да просто говорят - не мутируй передаваемый как аргумент объект

читай между строк)

Да похоже

Ну и в git чище коммиты с обязательной запятой, всегда ее ставлю.

согласен

давно пора делать cvs которая умеет в ast

да просто говорят - не мутируй передаваемый как аргумент объект

Не мутировать - понятно зачем. А манипулировать им почему нельзя? :)

bad example выглядит так

obj.key = 1;

Это понятно

Это как раз мутация

давно пора делать cvs которая умеет в ast

??

давно пора делать cvs которая умеет в ast

вот это было бы здорово) Плевать на код-стайл и прочую фигню. Только диффы ast - это диффы сериализующего формата, json, например)

esgit commit -am 'something great'

cst уже допилили?

Мозилла не опускают руки - запустили тестовый полигон для проверки фич, называется TestPilot. Это расширение, позволяющее протестировать новые тестовые фичи браузера. Главное что вам нужно знать про тестпилот - ТАБЫ СБОКУ * https://testpilot.firefox.com

Ахаха :)

Unstuck Webpack, интерфейс для настройки Webpack, результат можно посмотреть и скачать — http://www.linuxenko.pro/unstuck-webpack/

ы

http://reactkungfu.com/2016/03/dive-into-react-codebase-handling-state-changes/

ERROR: S client not available

https://github.com/kadirahq/react-storybook

народ, а подскажите, как в вебпаке переопределить зависимость?

вот какая-то библиотека подключает lodash/get, а я хочу чтобы подключала src/utils/get ?

пиши resolve.alias

мм, ну я пробую - не получается

как должен выглядить конфиг?

resolve: { alias: { get: 'src/utils/get', } }

Как вы код подкрашиваете?)

Как вы код подкрашиваете?)

как в markdown

далее просто import get from 'get'

кто-нибудь знает как в css modules импортировать весь файл как :global ?

завернуть весь контент файла в :global { }

кстати да, а получше вариант есть?

как вариант, добавлять постфикс .global.css и вешать еще один лоудер

https://github.com/chentsulin/electron-react-boilerplate/blob/master/webpack.config.development.js#L28 например

это уже лучше

это уже лучше

Дрочество.

а оборочивать в :global файл сторонней библиотеки нет?

а оборочивать в :global файл сторонней библиотеки нет?

В этом случае да, согласен, лодер логичное решение.

resolve: { alias: { get: 'src/utils/get', } }

внимательнее читаем вопрос) Мне нужно переопределить зависимость чужого модуля

внимательнее читаем вопрос) Мне нужно переопределить зависимость чужого модуля

Вместо get: пиши имя в том виде в котором сторонний модуль его подключает

а если это внутренняя зависимость?

по относительному пути типа ../get ?

например

хочу Link у RR переопределить

ну попробуй так же. Не уверен что прокатит правда Если "какая-то библиотека" подключает внешнюю зависимость по относительному пути, то это плохо написанная библиотека ) Если ты хочешь свой модуль подсунуть - то форкай RR :)

в общем ответ - ты не должен такого хотеть

странный ответ ;)

вообще странно, зачем подключать модуль с другой библиотеки, при этом самому переопредлять на свой)

странный ответ ;)

Вполне логичный, как по мне

Подсовывать депенденси для third party либы

Ну как-то это уж очень странно