ValentinSidorov / DeLorean_Team

Source project automotive BASECAMP
MIT License
0 stars 2 forks source link

Serhii shmahlii #12

Closed andrey-berdnik closed 2 years ago

andrey-berdnik commented 2 years ago

Pull request from Serhii Shmahlii

andrey-berdnik commented 2 years ago

По моему мнению твою ветку не стоит мерджить в основную, т.к. она не совпадает со структурой проекта (и это связанно с тем что ты не обновил свою ветку из main https://www.togaware.com/linux/survivor/Git_Merge_Master_into.html )

Так-же предлагаю твою задачу решить двумя файлами src/hello.cpp и src/include/hello.hpp

В ТЗ к твоей части работы (https://github.com/ValentinSidorov/DeLorean_Team/issues/7) я описал свое виденье как реализовать все пункты с использованием аргументов запуска (https://ravesli.com/urok-110-argumenty-komandnoj-stroki/), а не отдельных программ, это должно уменьшить количество оверхеда на решении этой задачи

Т.е. я предлагаю: слить код двух программ в одну, вписать его в существующую структуру проекта

zonmen commented 2 years ago

Полностью согласен с Андреем на счет мерджа, так как мэин у нас другой. И отличная идея с одним файлом для запуска и использлованием аргументов командной строки

ValentinSidorov commented 2 years ago