kDnMod firmware - #2
Conversation
…com/gunner47/GyverLamp и значительно переработана. Подробности спрашивайте на форуме.
… предыдущего режима
|
Все изменённые файлы должны оставаться в папке firmware/GyverLamp_v1.4, иначе невозможно понять, чем ваша версия отличается от оригинала. |
|
Даже здесь есть пример правильного PR: |
Изменения значительны. При желание можете попробовать по-файловое сравнение, хотя это мало поможет, т.к. структура проекта также частично изменена.
Это отдельный проект, отдельная ветка. Если вы хотите иначе - делайте самостоятельно в своем форке. Я в общем-то не настаиваю на мердже в мастер. Нужно - используйте, нет - игнорируйте. |
|
На всякий случай, обсуждение ведется здесь: https://community.alexgyver.ru/threads/wifi-lampa-budilnik-obsuzhdenie-proshivki-ot-gunner47.2418/ |
|
Просто pull request подразумевает именно мердж в оригинальный проект, а иначе зачем вообще делать PR? |
И? Я предлагаю мердж в основной проект, т.к. пока что текущая версия более-менее совместима (по еепром, порядку и кол-ву эффектов, приложению). Дальше в изменениях она уйдет перестанет быть таковой. Так исторически сложилось, что за базовый я взял именно вариант gunner47, поскольку он был на момент старта наиболее востребован. Повторюсь - PR можно реджектить, я совершенно спокойно к этому отнесусь и более не буду их создавать :) . |
…, чуть поправлен вывод новогоднего поздравления, а так же другие мелкие багфиксы
…енная), багфиксы вывода новогоднего поздравления, небольшое изменение setup для режима без кнопки.
3 participants
No description provided.