OpenVoiceOS / ovos-installer

Open Voice OS and/or HiveMind installer using Ansible with an intuitive and easy Text-based User Interface
https://www.openvoiceos.org
48 stars 12 forks source link

Handle scenario existing file but empty #72

Closed goldyfruit closed 2 months ago

goldyfruit commented 2 months ago

The installer should check if the scenario file contains YAML data as for now an empty scenario file triggers the ovos-installer to run non-interactive.

suvanbanerjee commented 2 months ago

@goldyfruit Can i work on this issue ?

I am thinking to create a validate.sh file that will use regex to validate the correctness of config.yaml

goldyfruit commented 2 months ago

I am thinking to create a validate.sh file that will use regex to validate the correctness of config.yaml

No need to add complexity, the scenario logic is already there, just few missing things. Have a look to the draft https://github.com/OpenVoiceOS/ovos-installer/pull/73, still have to fix the BATS testing.