search

bem / bem-bl

Base BEM library
http://bem.github.com/bem-bl/
198 stars 84 forks source link

Сommits in support/2.x which was missed to port into dev #623

Closed alexbaumgertner closed 9 years ago

alexbaumgertner commented 9 years ago

https://github.com/bem/bem-bl/pull/619

alexbaumgertner commented 9 years ago

Всегда делал релозв в пользу dev. Какой-то странный результат...

qfox commented 9 years ago

Ну как бы вообще не ожидаемый... Наверное, правильнее было бы cherry-pick сделать.

qfox commented 9 years ago

Хотя, есть такое мнение, что если в dev есть ВСЕ коммиты, а в support только часть, то вполне ожидаемый. Правда, видимо, там все коммиты, кроме этого ;-)

alexbaumgertner commented 9 years ago

@zxqfox сделал ручной вдумчивый resolve

qfox commented 9 years ago 
npm ERR! install Couldn't read dependencies
npm ERR! Failed to parse json
npm ERR! Unexpected token b
npm ERR! File: /home/travis/build/bem/bem-bl/package.json

Как-то не очень.

alexbaumgertner commented 9 years ago

@zxqfox ошибка была в package.json, поправил.

qfox commented 9 years ago

:+1: похоже на правду. Хорошо бы еще дифф посмотреть относительно 1.0 и 2.0.

alexbaumgertner commented 9 years ago

Надо посмотреть дифф относительно support/2.x, какие потенциально опасные коммиты есть в dev, так как версии потом будут на основе dev выпускаться.

alexbaumgertner commented 9 years ago

В общем, надо сделать ветку dev-without-support/2.x для проверки коммитов, которые никогда не релизились

qfox commented 9 years ago

Возможно, да. Я хочу это все посмотреть, но не смогу полноценно вдумчиво это сделать до след. недели. Если ты не торопишься, потерпим до след недели?

alexbaumgertner commented 9 years ago

@zxqfox это не только от меня зависит, мне нужно как можно быстрее :)

mishaberezin commented 9 years ago

Не, Саш, ты что-то не то здесь делаешь. Нужно отвести ветку от support/2.x, очень вдумчиво черипикнуть туда все полезное неломающее из dev и открыть PR в support/2.x.

В общем закрываю, будет другой PR.