Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Объединение веток dev и support/2.x #619

Closed
wants to merge 20 commits into from

Conversation

alexbaumgertner
Copy link
Member

BEM-2166: bem-bl: is it possible to merge dev and support/2.x?

  1. tree support-to-dev/2.x base on support/2.x

@alexbaumgertner
Copy link
Member Author

  1. rebase support-to-dev/2.x on dev

One error occured: 1) i-bem block and others mix with same block bem-core/804:

extra class b1

@alexbaumgertner
Copy link
Member Author

  1. rebase support-to-dev/2.x on v2.7.0 (last release)

@qfox
Copy link
Member

qfox commented Mar 12, 2015

А мастер ветка кому-то нужна? может туда перенести? Или я не понял идеи?

@alexbaumgertner
Copy link
Member Author

Закрыть support/2.x и разрабатывать в dev-ветке.
suppport/1.x оставить для поддержки версии 1

/cc @tadatuta

@alexbaumgertner
Copy link
Member Author

https://github.com/bem/bem-bl/commits/master надо похоже вообще удалить, как устаревшую.

@qfox
Copy link
Member

qfox commented Mar 12, 2015

Вот и я думаю, что можно зарезать мастер, дев, оставить support/1.x, и support/2.x унести в master вместе с нужными коммитами из текущего dev. Если это реально, конечно ;-)

@tadatuta
Copy link
Member

@alexbaumgertner ты же не всерьез это, я надеюсь? 😨

@alexbaumgertner
Copy link
Member Author

@tadatuta нет :) Это для исследования.
Но нужно продумать алгоритм объединения веток dev и support/2.x.
Как они различаются? dev просто ушла вперед от support/2.x? Расскажи пожалуйста, как появились эти две ветки?

@tadatuta
Copy link
Member

@alexbaumgertner dev — это ветка, куда попадали все полезные фичи, а затем нужные в конкретных support/* портировались через cherry-pick.
из того, что точно отличается — это документация и сборка. сейчас на bem.info и Лего-сайт собирается только dev-ветка. потенциально в dev могут быть коммиты, ломающие обратную совместимость, недотестированные и все такое. кроме того, возможна ситуация, что в support/* могли попасть коммиты, которые не попали в dev.
в общем, это задача на вдумчивое ручное портирование, а не просто про ребейз.

@alexbaumgertner
Copy link
Member Author

Как я понимаю, сейчас в dev ветке есть коммиты и из support/1.x и из support/2.x?
Надо быть сильно в контексте, чтобы понять, как правильно коммиты разрулить.
Похоже, я не справлюсь :(

@alexbaumgertner
Copy link
Member Author

Определились со стратегией:

  • Отвести от support/2.x ветку, допустим support-to-dev/2.x;
  • Найти общий коммит new-dev с dev;
  • Аккуратно cherry-pick коммиты из dev в new-dev, проверяя сборку тестов.
  1. Сейчас support-to-dev/2.x = support/2.x

  2. Нашел общий коммит с dev, 47a261:

tadatuta authored on Apr 28, 2014
Merge pull request #506 from bem/test@2.x
Bulk PRs merge
i-bem__dom: Store block params in data attributes
i-bem__dom: Fix for blocks without domElem
BEMHTML improvements

@alexbaumgertner
Copy link
Member Author

Нашел все коммиты в dev, которых нет в support/2.x и наоборот.
git log --cherry-mark --pretty support-to-dev/2.x...dev
https://gist.github.com/alexbaumgertner/3ffc3f72f5612aff4a6d
(знак =значит коммиты есть в обоих ветках)

Sergey Tatarintsev and others added 3 commits March 24, 2015 23:14
Live initialization in `_interaction_yes` modifier prevents some
other modifier (i.e. `platform`, `browser` etc) from being set during
initialization. Some CSS selectors match on them and if used along with
`i-ua_intearaction_yes` it causes page to appear differently before and
after click.
The way it was written it was impossible to override `b-page` mixes
on further levels.
@alexbaumgertner
Copy link
Member Author

@tadatuta нашел все коммиты в support/2.x, которых нет в dev.
Положил их в support-to-dev/2.x
Сделал этот PR.

Вот вариант с резолвом конфликтов #623

@mishaberezin
Copy link
Member

sorry guys

@mishaberezin mishaberezin deleted the support-to-dev/2.x branch April 1, 2015 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants