vk
ребята, не помните, кто тут писал про то, что у него питонячего кода на проекте - 20к файлов?
Size: a a a
2020 May 30
SD
да, получается через рефакторинг, хотя что это значит я не очень понимаю
рефакторинг в пайчарме это типа умный рефакторинг, который на деле не такой уж и умный.
А рефакторинг — опять же в понимании pycharm — это изменение кода (упрощенно)
А рефакторинг — опять же в понимании pycharm — это изменение кода (упрощенно)
i
рефакторинг в пайчарме это типа умный рефакторинг, который на деле не такой уж и умный.
А рефакторинг — опять же в понимании pycharm — это изменение кода (упрощенно)
А рефакторинг — опять же в понимании pycharm — это изменение кода (упрощенно)
т.е. операция с допущением на изменение кода в последствии. ясно, спасибо)
DD
victor kurguzov
а могут не пропустить ПР если в нем, скажем, есть лишние отступы между строками, где-то 2, где-то 1?
Кто? ПР?
vk
Кто? ПР?
пулл реквест
DD
Понял
vk
меня на днях сильно выбесил один из ваших, больше 20 комментариев к 1 ПРу и все на тему нейминга и форматирования
SD
victor kurguzov
а могут не пропустить ПР если в нем, скажем, есть лишние отступы между строками, где-то 2, где-то 1?
мы работаем без пр: если задача зарезолвлена в джире, значит можно ревьюить.
Лишние отступы уберет ревьюер, или ответсвенный за деплой. Если их немного, конечно. Но такого, чтобы в закоммиченном коде было много ненужных пустых строк, думаю не бывало.
Лишние отступы уберет ревьюер, или ответсвенный за деплой. Если их немного, конечно. Но такого, чтобы в закоммиченном коде было много ненужных пустых строк, думаю не бывало.
SD
victor kurguzov
меня на днях сильно выбесил один из ваших, больше 20 комментариев к 1 ПРу и все на тему нейминга и форматирования
с форматированием и неймингом та же история. Если что, то такие вещи может пофиксить ревьюер (это гораздо быстрее, чем переоткрывать задачу).
Хотя бывало, что меня просили переименовать какой-нибудь класс, функцию или переменную
Хотя бывало, что меня просили переименовать какой-нибудь класс, функцию или переменную
vk
мы работаем без пр: если задача зарезолвлена в джире, значит можно ревьюить.
Лишние отступы уберет ревьюер, или ответсвенный за деплой. Если их немного, конечно. Но такого, чтобы в закоммиченном коде было много ненужных пустых строк, думаю не бывало.
Лишние отступы уберет ревьюер, или ответсвенный за деплой. Если их немного, конечно. Но такого, чтобы в закоммиченном коде было много ненужных пустых строк, думаю не бывало.
в смысле без пр? с одним мастером что ли?
SD
т.е. когда я говорю что без ПР, то имею в виду что этот процесс не оформлен в виде некоего интерфейса (как на гитхабе например).
Если резолв задачи в джире в качестве сигнала о том, что ветка готова к ревью (и возможно к деплою), можно назвать пулл реквестом, то тогда да, у нас есть такие "пулл реквесты"
Если резолв задачи в джире в качестве сигнала о том, что ветка готова к ревью (и возможно к деплою), можно назвать пулл реквестом, то тогда да, у нас есть такие "пулл реквесты"
vk
т.е. когда я говорю что без ПР, то имею в виду что этот процесс не оформлен в виде некоего интерфейса (как на гитхабе например).
Если резолв задачи в джире в качестве сигнала о том, что ветка готова к ревью (и возможно к деплою), можно назвать пулл реквестом, то тогда да, у нас есть такие "пулл реквесты"
Если резолв задачи в джире в качестве сигнала о том, что ветка готова к ревью (и возможно к деплою), можно назвать пулл реквестом, то тогда да, у нас есть такие "пулл реквесты"
теперь понятно, спасибо
SD
victor kurguzov
меня на днях сильно выбесил один из ваших, больше 20 комментариев к 1 ПРу и все на тему нейминга и форматирования
а недавно даже попросили выражение вида "a, b = x, y" привести к виду "a, b = (x, y)" 😀
Я был не очень согласен с эстетической точки зрения, но стайлгайд есть стайлгайд (даже если он нигде не зафиксирован)
Я был не очень согласен с эстетической точки зрения, но стайлгайд есть стайлгайд (даже если он нигде не зафиксирован)