intents
intents copied to clipboard
home-assistant
bg language edit
Do you pull to the current code state before making any code changes and PRs? Do you merge this main branch in to yours? I'see you're committing the same code, that I redacted yesterday, again and again with the same mistakes. This constantly creates conflicts, that has to be resolved manualy.
First RP was closed from you … then I some how mess it up because I receive a lot of errors. Then I just delete the branch and creat this one with fix on all mistake that you was point first. When I post this RP it was merge with the main. Last night you skip this new RP and start to work on my old branch that I deleted and it was closed from you. Then now the RP look strange. Just have so much thing to be edited. But first don’t blame me for the new branch. For me close it mean close. Hope you see language.yaml file now. But let’s close that fight. I really want to make everything that is need to can make voice control on Bulgarian true. Let’s make one proper common file with all words in it , in action files to be only code. If you check all the commits that I did to now on this branch you will get what I mean. Cann all commits from me to be merge in one , to look how is look now in my code line. this way i agree is messi.
Я дай на български, че почнах леко да изпушвам. Файлът с езиците аз ти го редактирах и мърджах. Всъщност аз на твоите 3 PR-a направих 7, докато го докато го издокарам да работи. Второ отхвърлих го, но това не значи да не си обновяваш кода в твоето репозитори с оригиналния, който е на хоум асистант. Трето вписваш прекалено много неща и в common и във fixtures, които са само за тест, а не за пълен превод и какво ли още не. Да, може някой ден да са от полза, ако решат да ги използват за тази цел. В момента това са просто тестови данни (визирам fixтures), за да се тестват изреченията. За нищо друго. Когато работиш с git в екип, винаги трябва да си опресняваш твоето repo особено след PR-и мърджове и какво ли не в главното (там от където си копирал твоето). Не може да изравним на HA репозиторито с твоето, защото твоето съдържа грешки. Трбва след всеки отказан, одобрен, мърджнат и какво ли още не PR ти да си изравниш твоя код с кода от ХА, понеже не само ти правиш PR-и. Kaкто го правиш в момента, затриваш на другите промените (или поне се опитваш, понеже аз не ги одобрявам) ..
Смятам знаеш, че съм го направил, за да не ти чупя ентусиазма, понеже не съм тук за да се заяждам. Знаеш, че за да получиш CLA трябваше да имаш поне един одобрен PR в репо-то. За това си играх аз да оправям твоите промени, за да може ти да го получиш.
Hey @BossNeo I would suggest you do smaller PRs. You have too many changes to review easily, there are allot of syntactic errors, merge conflicts and so on.
You have great additions, its just a bit sloppy atm. It takes quite some time to review and clean up. Last PR @hristo-atanasov spent time to cleanup.
Keep up the good work.
Пак казвам. Когато го пусах беше ъбдейтнат с главния. Твоите промени по стария бранч дойдоха след като го бях пуснал. Приемането на стария стана след катого бях изтрил - не си от скоро тука надали не си видял че е "бездомен". Видях какви промени си правил на него и ги приложих и тука. Именно затова изтрих стария бранч защото спрях да виждам какъв е основния почнаха да излизат грешки и го и го изтрих и почнах на чисто. Първо чупи стойки затвори стария философстваше как трябва да е изписано български в language.yml, но няой умен го е поправил. Сега не си... Разбирам че са много файлове ,но за да проверя дали работи трябва да едитна друг файл , който води до 3 се насъбират. Последното беше за да направя "Запали кухненските лапи" да работи . се пипат 5 файла , затова искам да изкараме всички иьрази в common - да стават много , но е по удобно за добавяне на думи стига структурата в другите файлове да е добра. А именно за да не стават много промени не съм почнал щорите.
Да разбирам ви. Оправете си бранча да се мърджва чисто с main и ще можем да го разгледаме. Хубаво е и да пуснете тестовете локално, че ме съмняват някои неща.
I understand. Please fix your branch so it cleanly merges into main and we can discuss further. It would be great if you could also run tests locally, as I think some things look broken.
Първо чупи стойки затвори стария философстваше как трябва да е изписано български в language.yml, но няой умен го е поправил.
Eeee .. Имаме напредък .. Това за комплимент ли да го считам? :D
PS: Нищо друго от написаното не разбрах. Ама ще се оправим все някак. Виж какъв съвет ти е дал @dobromir-hristov и поне него послушай. :) Знам, че когато напишем нещо тук, изглежда все едно група снайперисти те дебне да те елиминира, но всъщност не е така. Дори се опитвам да спазвам благоприятен тон, при все че не ми се отдава много (и си личи). :)
Да разбирам ви. Оправете си бранча да се мърджва чисто с
mainи ще можем да го разгледаме. Хубаво е и да пуснете тестовете локално, че ме съмняват някои неща.I understand. Please fix your branch so it cleanly merges into
mainand we can discuss further. It would be great if you could also run tests locally, as I think some things look broken.
няма проблеми, стига да разбра как
I'm closing this PR as the recent one was approved. If you still find some parts of the code useful, you can still copy it in your code and include it in your next PR. Also one important notice - next week, between 16.01 and 22.01, I'll be on a business trip outside of the country and I'll handle new PRs when I come back. :)