будете ругаться или спасибо скажете..

[starkiller]

..если коллега возьмет ваш код из [dev branch] вместе со своим, прогонит все необходимые тесты и зачекинит под своим логином?

[thinker]

У нас нет понятия "ваш код" или "мой код". Все что вы пишите для компании принадлежит этой компании. И когда один девелопер берeт чужой код а потом его модифицирует, добавляет фичи - это каждодневная норма.

[starkiller]

Да, но все таки приятно оставить след в истории в виде очередного коммита в гит логе. Тем более в данном случае проект был нетривиальный в отличие от каждодневного багфиксинга. Хотя здравый смысл подсказывает что главное что бы начальник знал кто что сделал.

[thinker]

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

[Easbayguy]

Да, но все таки приятно оставить след в истории в виде очередного коммита в гит логе. Тем более в данном случае проект был нетривиальный в отличие от каждодневного багфиксинга. Хотя здравый смысл подсказывает что главное что бы начальник знал кто что сделал.

Напишите е-маил начальнику и етому перцу, спросите является ли теперь он владельцем украденного кода.

[mynameiszb]

Напишите е-маил начальнику и етому перцу, спросите является ли теперь он владельцем украденного кода.

Фиг с ним - "владельцем". Главное - кто теперь это саппортить будет и баги закрывать
А то как чужой код за свой выдавать - так очередь желающих. А как потом это сопровождать - прибегают в слезах и соплях и требуют (о как даже - требуют!), чтобы все бросил и "в твоей гадости" копался...

[rzen]

Обязательно поставить в известность начальнику. Если не в первый же раз то уж в случае рецидива обязательно.

[kostik78]

Если честно то я не понял как история чекина в этом случае потеряется если это GIT. Вы сделали сабмит в dev-branch, коллега сделал pull ваших изменений - все по идее его git log уже имеет историю о Ваших изменения и при следующем push попадет туда куда надо unless коллега занимается читерством и чистить git log.

[crypto5]

Обязательно поставить в известность начальнику. Если не в первый же раз то уж в случае рецидива обязательно.

Можно прослыть не team player, склочником и т.д. Это часть гонок.

[rzen]

Обязательно поставить в известность начальнику. Если не в первый же раз то уж в случае рецидива обязательно.

Можно прослыть не team player, склочником и т.д. Это часть гонок.

ну не обязательно вот так прям строчить кляузы можно мирно культурно в разговоре в присутствии свидетелей упомянуть мол смотри какой Боб умница, взял мой непричесанный код, вылизал и теперь вона как здорово получилось.

другой вопрос у босса на это может оказаться другой взгляд и особенно если этот боб ворует код частенько значит он скорее всего нахлебничает в команде. а зачем вам такой "тим плеер"? а уж боссу и подавно.

[crypto5]

Обязательно поставить в известность начальнику. Если не в первый же раз то уж в случае рецидива обязательно.

Можно прослыть не team player, склочником и т.д. Это часть гонок.

ну не обязательно вот так прям строчить кляузы можно мирно культурно в разговоре в присутствии свидетелей упомянуть мол смотри какой Боб умница, взял мой непричесанный код, вылизал и теперь вона как здорово получилось.

другой вопрос у босса на это может оказаться другой взгляд и особенно если этот боб ворует код частенько значит он скорее всего нахлебничает в команде. а зачем вам такой "тим плеер"? а уж боссу и подавно.

Ну я таким артистизмом не владею, а боссу скорее всего пофиг ))

[rzen]

Обязательно поставить в известность начальнику. Если не в первый же раз то уж в случае рецидива обязательно.

Можно прослыть не team player, склочником и т.д. Это часть гонок.

ну не обязательно вот так прям строчить кляузы можно мирно культурно в разговоре в присутствии свидетелей упомянуть мол смотри какой Боб умница, взял мой непричесанный код, вылизал и теперь вона как здорово получилось.

другой вопрос у босса на это может оказаться другой взгляд и особенно если этот боб ворует код частенько значит он скорее всего нахлебничает в команде. а зачем вам такой "тим плеер"? а уж боссу и подавно.

Ну я таким артистизмом не владею, а боссу скорее всего пофиг ))

зависит от шопа конечно но врядли уж совсем пофиг. вот вы как ценный работник уйдёте на лУчшие хлеба, он останется с этим виртуозом которому код копировать будет неоткуда. производительность упадёт на 100%.

[crypto5]

Обязательно поставить в известность начальнику. Если не в первый же раз то уж в случае рецидива обязательно.

Можно прослыть не team player, склочником и т.д. Это часть гонок.

ну не обязательно вот так прям строчить кляузы можно мирно культурно в разговоре в присутствии свидетелей упомянуть мол смотри какой Боб умница, взял мой непричесанный код, вылизал и теперь вона как здорово получилось.

другой вопрос у босса на это может оказаться другой взгляд и особенно если этот боб ворует код частенько значит он скорее всего нахлебничает в команде. а зачем вам такой "тим плеер"? а уж боссу и подавно.

Ну я таким артистизмом не владею, а боссу скорее всего пофиг ))

зависит от шопа конечно но врядли уж совсем пофиг. вот вы как ценный работник уйдёте на лУчшие хлеба, он останется с этим виртуозом которому код копировать будет неоткуда. производительность упадёт на 100%.

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

[Easbayguy]

Вас поимели, ваши дейстивя?

- Что то сделать, написать начальству.
- Утереться, потому что чего то боитесь, не умеете писать письма, русский интелегент.

[rzen]

Можно прослыть не team player, склочником и т.д. Это часть гонок.

ну не обязательно вот так прям строчить кляузы можно мирно культурно в разговоре в присутствии свидетелей упомянуть мол смотри какой Боб умница, взял мой непричесанный код, вылизал и теперь вона как здорово получилось.

другой вопрос у босса на это может оказаться другой взгляд и особенно если этот боб ворует код частенько значит он скорее всего нахлебничает в команде. а зачем вам такой "тим плеер"? а уж боссу и подавно.

Ну я таким артистизмом не владею, а боссу скорее всего пофиг ))

зависит от шопа конечно но врядли уж совсем пофиг. вот вы как ценный работник уйдёте на лУчшие хлеба, он останется с этим виртуозом которому код копировать будет неоткуда. производительность упадёт на 100%.

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

это конечно не радикальный случай но смысл тот же: all it takes for the evil to triumph is for good men to do nothing. вы промолчали, кто то промолчал, и вот мы все дружно кормим очередного нахлебника.

я ещё не видел ни одного толкового программиста кто а) не крал бы код и б) при этом обязательно не прописывал бы откуда взято.

[crypto5]

Вас поимели, ваши дейстивя?

- Что то сделать, написать начальству.
- Утереться, потому что чего то боитесь, не умеете писать письма, русский интелегент.

Иметь мозг начальства по мелочам обычно себе дороже. Лучше оставить депозит доверия на более серьезные вещи.

[crypto5]

Вас поимели, ваши дейстивя?

- Что то сделать, написать начальству.
- Утереться, потому что чего то боитесь, не умеете писать письма, русский интелегент.

Иметь мозг начальства по мелочам обычно себе дороже. Лучше оставить депозит доверия на более серьезные вещи.

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

[Интеррапт]

..если коллега возьмет ваш код из [dev branch] вместе со своим, прогонит все необходимые тесты и зачекинит под своим логином?

Ну напишите письмо с вопросами (включая начальника), насколько детально проверил он проверил код, потому что вы еще собственное ревью кода не делали и опасаетесь, что там может быть куча косяков.

[asktomsk]

Вообще так можно сделать когда используется git rebase вместе со squash. Т.е. некоторые коммиты схлопываются в один, потяно, что кое-что авторство пропадает.

Есть одна замечательная статья на хабре про эту тему: habrahabr.ru/post/179123/

Автору могу посоветовать тихонько подсадить воришку на свои коммиты, а потом сделать из него "Васю" из статьи

[starkiller]

Если честно то я не понял как история чекина в этом случае потеряется если это GIT. Вы сделали сабмит в dev-branch, коллега сделал pull ваших изменений - все по идее его git log уже имеет историю о Ваших изменения и при следующем push попадет туда куда надо unless коллега занимается читерством и чистить git log.

Если бы он сделал pull/push то не было бы и вопросов. Он просто взял все файлы и сделал из них новый коммит. Типа был слишком мутный мердж.

[starkiller]

Все сразу стали говорить про украл код, а вопрос про спасибо в сабдже был не иронией. Я думaю может действительно человек за дело болеет, а чья там подпись под "чертежом" дело десятое. Тесты прогнал все таки. Может моя реакция просто неадекватна. Просто мне всегда казалось это нарушением этикета что-ли. Вот и хотел узнать у других как принято. Может в порядке вешей.

[Сабина]

Если честно то я не понял как история чекина в этом случае потеряется если это GIT. Вы сделали сабмит в dev-branch, коллега сделал pull ваших изменений - все по идее его git log уже имеет историю о Ваших изменения и при следующем push попадет туда куда надо unless коллега занимается читерством и чистить git log.

Если бы он сделал pull/push то не было бы и вопросов. Он просто взял все файлы и сделал из них новый коммит. Типа был слишком мутный мердж.

А я так думаю подобный "мутный мердж" только от большого незнания может получится. Вообще то принято делать апдейт хотя бы раз в день, особенно на проекте где много чекинят.

[starkiller]

Апдейт чего? Дев бранча?

[Сабина]

Апдейт чего? Дев бранча?

Локального workspace из репозитория

[Интеррапт]

Если честно то я не понял как история чекина в этом случае потеряется если это GIT. Вы сделали сабмит в dev-branch, коллега сделал pull ваших изменений - все по идее его git log уже имеет историю о Ваших изменения и при следующем push попадет туда куда надо unless коллега занимается читерством и чистить git log.

Если бы он сделал pull/push то не было бы и вопросов. Он просто взял все файлы и сделал из них новый коммит. Типа был слишком мутный мердж.

А я так думаю подобный "мутный мердж" только от большого незнания может получится. Вообще то принято делать апдейт хотя бы раз в день, особенно на проекте где много чекинят.

Для git - далеко не обязательно.