search

caenHV / caen_tools

Set of microservices for CAEN management (WebService, DeviceBackend, SystemCheck, Monitor)
0 stars 0 forks source link

Implementation of Popov's requests (issue #26) #27

Closed AliensWithdrawPeople closed 3 weeks ago

AliensWithdrawPeople commented 3 weeks ago

Тут всё, что я написал в связи с issue. Там помимо добавления были некоторые изменения, связанные с shared_parameters для HealthControl. Сам релиз уже сделан (v2.3.12). На железе стоит по факту эта версия, хоть и называется v2.3.12-beta1. Тесты без пучков показали работоспособность, но как будет с пучками не понятно.

Чекни, пожалуйста, код на предмет ляпов, которые тесты не выявили, и пробелов в логике.

AliensWithdrawPeople commented 3 weeks ago

Коммитов много, но почти все муссорные. Сорян за захламление ветки.

nikita-p commented 3 weeks ago

если всё работает, то отлично, ведь "практика - критерий истины" (я по диагонали посмотрел код, вроде всё хорошо)

Коммитов много, но почти все муссорные

тогда заксвошу изменения в один коммит в dev