Break up visplay.yaml - Githubissues

ColoradoSchoolOfMines / visplay

A libmpv and Qt based application for automating and controlling media playback on any display.

https://coloradoschoolofmines.github.io/visplay/

GNU General Public License v3.0

4 stars 1 forks source link

Break up visplay.yaml #6

Closed robozman closed 6 years ago

robozman commented 6 years ago

I think it would be a good idea to break up visplay.yaml into visplay.yaml, for general configuration and sources.yaml for locations of assets/sources/playlists.yaml.

Thoughts?

jhgarner commented 6 years ago

That sounds good and should be extremely easy since it would just be a rename. Visplay.config doesn't have support for anything but sources.

robozman commented 6 years ago

Great! I think it will also make https://github.com/ColoradoSchoolOfMines/visplay/issues/5 easier as we will not have to worry about ignoring non source related settings when obtained recursivly.

robozman commented 6 years ago

Requirements

[x] update visplay to look at the correct files https://github.com/ColoradoSchoolOfMines/visplay/commit/3ee1eb29487107b50daebcc49f65593ce3824098
[x] make *.yaml.example files https://github.com/ColoradoSchoolOfMines/visplay/commit/ddb01446fb1a37195834b2ec406bd60e7a3400c7
[x] add new files to .gitignore https://github.com/ColoradoSchoolOfMines/visplay/commit/e8d10858993c3fd4db08893d83251f26948d5893
[x] update configuration creation instructions in the README https://github.com/ColoradoSchoolOfMines/visplay/commit/d1df850b7187e229dcdd9a4b28f8c6939ad67f51
[ ] document configuration format

robozman commented 6 years ago

I'm breaking visplay,yaml into config.yaml and sources.yaml. Better naming imo

robozman commented 6 years ago

@jhgarner The README needs to be updated to your new config syntax

robozman commented 6 years ago

This seems to be finished. We have settled on main entrypoint config.yaml, and sources.yaml files that can point to assets/playlists.yaml and import other sources.