Чуть переработанный импорт файлов

Форум для обсуждения деталей разработки программы SAS.Планета

Модераторы: vdemidov, Tolik

Robbi
Новичок
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45

Re: Чуть переработанный импорт файлов

Сообщение Robbi »

garl писал(а):от какой ревизи (числа) форк? а то у меня изменений много.
Я постоянно обновляю репозиторий, свои изменения держу в отдельной ветке (от 7262 начал, а так в рабочей копии всё лежало и мёрджилось при новых обновлениях опять в рабочую) а тестирую после слияния с основной.
на 7269 (f05f1f58d148) (последнее что вытянулось) патч нормально ложится и работает.
zed
Гуру
Сообщения: 2888
Зарегистрирован: 16 авг 2008, 20:21
Благодарил (а): 89 раз
Поблагодарили: 568 раз

Re: Чуть переработанный импорт файлов

Сообщение zed »

Robbi писал(а):Не очень представляю как сделать пуш только перенесенной в основную ветку заплатки, без своей ветки изменений. Вот заплатка, если посчитают достойным её добавление =)
Решение в лоб: делаете новый форк, накладываете на него патч и делаете пул реквест.
Robbi
Новичок
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45

Re: Чуть переработанный импорт файлов

Сообщение Robbi »

zed писал(а):
Robbi писал(а):Не очень представляю как сделать пуш только перенесенной в основную ветку заплатки, без своей ветки изменений. Вот заплатка, если посчитают достойным её добавление =)
Решение в лоб: делаете новый форк, накладываете на него патч и делаете пул реквест.
Уже нашёл hg push -b или -r для ветки и ревизии соответственно.
Сейчас переделываю тултип - вместо текста отображение всего содержимого метки - удобнее смотреть фотки на карте. Как закончу - сделаю пул реквест
zed
Гуру
Сообщения: 2888
Зарегистрирован: 16 авг 2008, 20:21
Благодарил (а): 89 раз
Поблагодарили: 568 раз

Re: Чуть переработанный импорт файлов

Сообщение zed »

И хорошо бы по каждому пункту доработки открыть тикеты в багтреке, потому что нужно конкретно смотреть на суть каждой доработки и её реализацию.
Robbi
Новичок
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45

Re: Чуть переработанный импорт файлов

Сообщение Robbi »

zed писал(а):И хорошо бы по каждому пункту доработки открыть тикеты в багтреке, потому что нужно конкретно смотреть на суть каждой доработки и её реализацию.
Быть может тогда залить всю мою ветку с описанием реализованных функций в виде тикетов и если что - потом перенесут нужные коммиты в основную ветку?
Аватара пользователя
garl
Гуру
Сообщения: 1625
Зарегистрирован: 16 июл 2008, 14:40
Откуда: Краснодар, Кубанская столица.
Благодарил (а): 97 раз
Поблагодарили: 272 раза

Re: Чуть переработанный импорт файлов

Сообщение garl »

лучше каждый тикет отдельным реквестом делать
так и понятнее и логичнее
к тому же реквесты принимаются не всегда сразу :)
Russian NDN Team
QIP NightlyTester
Robbi
Новичок
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45

Re: Чуть переработанный импорт файлов

Сообщение Robbi »

garl писал(а):лучше каждый тикет отдельным реквестом делать
так и понятнее и логичнее
к тому же реквесты принимаются не всегда сразу :)
Ясно) Ну в первом коммите все равно сразу два изменения - писал для себя, поэтому сразу не разбил.
Сейчас буду разбираться как пул реквесты пошагово сделать =) Создам только тикеты для них
zed
Гуру
Сообщения: 2888
Зарегистрирован: 16 авг 2008, 20:21
Благодарил (а): 89 раз
Поблагодарили: 568 раз

Re: Чуть переработанный импорт файлов

Сообщение zed »

Robbi писал(а): и если что - потом перенесут нужные коммиты в основную ветку
Маловероятно, что кто-то будет этим заниматься - переносить. А вот если прилетят 4-5 пул-реквестов, то велики шансы, что какие-то из них (или даже все) войдут в основную ветку.
Robbi
Новичок
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45

Re: Чуть переработанный импорт файлов

Сообщение Robbi »

zed писал(а):
Robbi писал(а): и если что - потом перенесут нужные коммиты в основную ветку
Маловероятно, что кто-то будет этим заниматься - переносить. А вот если прилетят 4-5 пул-реквестов, то велики шансы, что какие-то из них (или даже все) войдут в основную ветку.
Ну чтож, как я понял надо сделать форк, pull на комп, commit, push на сервер и потом pull request?
zed
Гуру
Сообщения: 2888
Зарегистрирован: 16 авг 2008, 20:21
Благодарил (а): 89 раз
Поблагодарили: 568 раз

Re: Чуть переработанный импорт файлов

Сообщение zed »

Robbi писал(а):Ну чтож, как я понял надо сделать форк, pull на комп, commit, push на сервер и потом pull request?
Да, всё верно.
Ответить

Вернуться в «Раздел для разработчиков программы SAS.Планета»