SASGIS

Веб-картография и навигация

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

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

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

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

Сообщение Robbi » 06 июн 2013, 12:47

garl писал(а):от какой ревизи (числа) форк? а то у меня изменений много.

Я постоянно обновляю репозиторий, свои изменения держу в отдельной ветке (от 7262 начал, а так в рабочей копии всё лежало и мёрджилось при новых обновлениях опять в рабочую) а тестирую после слияния с основной.
на 7269 (f05f1f58d148) (последнее что вытянулось) патч нормально ложится и работает.
Robbi
Новичок
 
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45
Благодарил (а): 0 раз.
Поблагодарили: 0 раз.

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

Сообщение zed » 06 июн 2013, 14:34

Robbi писал(а):Не очень представляю как сделать пуш только перенесенной в основную ветку заплатки, без своей ветки изменений. Вот заплатка, если посчитают достойным её добавление =)

Решение в лоб: делаете новый форк, накладываете на него патч и делаете пул реквест.
zed
Гуру
 
Сообщения: 2888
Зарегистрирован: 16 авг 2008, 20:21
Благодарил (а): 89 раз.
Поблагодарили: 525 раз.

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

Сообщение Robbi » 06 июн 2013, 14:41

zed писал(а):
Robbi писал(а):Не очень представляю как сделать пуш только перенесенной в основную ветку заплатки, без своей ветки изменений. Вот заплатка, если посчитают достойным её добавление =)

Решение в лоб: делаете новый форк, накладываете на него патч и делаете пул реквест.

Уже нашёл hg push -b или -r для ветки и ревизии соответственно.
Сейчас переделываю тултип - вместо текста отображение всего содержимого метки - удобнее смотреть фотки на карте. Как закончу - сделаю пул реквест
Robbi
Новичок
 
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45
Благодарил (а): 0 раз.
Поблагодарили: 0 раз.

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

Сообщение zed » 06 июн 2013, 14:57

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

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

Сообщение Robbi » 06 июн 2013, 15:16

zed писал(а):И хорошо бы по каждому пункту доработки открыть тикеты в багтреке, потому что нужно конкретно смотреть на суть каждой доработки и её реализацию.

Быть может тогда залить всю мою ветку с описанием реализованных функций в виде тикетов и если что - потом перенесут нужные коммиты в основную ветку?
Robbi
Новичок
 
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45
Благодарил (а): 0 раз.
Поблагодарили: 0 раз.

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

Сообщение garl » 06 июн 2013, 15:23

лучше каждый тикет отдельным реквестом делать
так и понятнее и логичнее
к тому же реквесты принимаются не всегда сразу :)
Russian NDN Team
QIP NightlyTester
Аватара пользователя
garl
Гуру
 
Сообщения: 1625
Зарегистрирован: 16 июл 2008, 14:40
Откуда: Краснодар, Кубанская столица.
Благодарил (а): 97 раз.
Поблагодарили: 245 раз.

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

Сообщение Robbi » 06 июн 2013, 15:26

garl писал(а):лучше каждый тикет отдельным реквестом делать
так и понятнее и логичнее
к тому же реквесты принимаются не всегда сразу :)


Ясно) Ну в первом коммите все равно сразу два изменения - писал для себя, поэтому сразу не разбил.
Сейчас буду разбираться как пул реквесты пошагово сделать =) Создам только тикеты для них
Robbi
Новичок
 
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45
Благодарил (а): 0 раз.
Поблагодарили: 0 раз.

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

Сообщение zed » 06 июн 2013, 15:28

Robbi писал(а): и если что - потом перенесут нужные коммиты в основную ветку

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

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

Сообщение Robbi » 06 июн 2013, 15:51

zed писал(а):
Robbi писал(а): и если что - потом перенесут нужные коммиты в основную ветку

Маловероятно, что кто-то будет этим заниматься - переносить. А вот если прилетят 4-5 пул-реквестов, то велики шансы, что какие-то из них (или даже все) войдут в основную ветку.

Ну чтож, как я понял надо сделать форк, pull на комп, commit, push на сервер и потом pull request?
Robbi
Новичок
 
Сообщения: 14
Зарегистрирован: 30 янв 2010, 20:45
Благодарил (а): 0 раз.
Поблагодарили: 0 раз.

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

Сообщение zed » 06 июн 2013, 16:01

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

Да, всё верно.
zed
Гуру
 
Сообщения: 2888
Зарегистрирован: 16 авг 2008, 20:21
Благодарил (а): 89 раз.
Поблагодарили: 525 раз.

Пред.След.

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

Кто сейчас на конференции

Сейчас этот форум просматривают: нет зарегистрированных пользователей и гости: 2