WIP: Expose server configuration to Wilfred

wilfred-dev / wilfred

🐿️ A CLI for managing game servers using Docker.

https://wilfredproject.org

MIT License

35 stars 2 forks source link

WIP: Expose server configuration to Wilfred #27

Closed vilhelmprytz closed 4 years ago

vilhelmprytz commented 4 years ago

Exposes server configuration to Wilfred.

Fixes #17 Fixes #21

Requires image API version 2, see images/#1.

TODO after merge: Update documentation accordingly

lgtm-com[bot] commented 4 years ago

This pull request introduces 3 alerts when merging 43b334b33ef67929fdc5038bc8924d9f57f2fff8 into f33a2e107170c259b125226e599cd944544e8d86 - view on LGTM.com

new alerts:

1 for Unnecessary pass
1 for Unused local variable
1 for Unused import

vilhelmprytz commented 4 years ago

parsing YAML in Wilfred

YAML files are now parsed properly. Nested dictionaries are separated using /. No ability to edit lists. Does not write YAML yet.

vilhelmprytz commented 4 years ago

Okay, so now .properties files are properly parsed (including the ability to edit settings). .yml files are only readable by Wilfred, not editable.

lgtm-com[bot] commented 4 years ago

This pull request introduces 2 alerts when merging 828a1406b49862e41f28606f40ef47cca44406c5 into 7174a3e266560c0f1e14dd34f387074e08214c6b - view on LGTM.com

new alerts:

2 for Unreachable code

vilhelmprytz commented 4 years ago

.properties are fully editable and writable. .yaml and .json are both fully readable, but not writable. Will work on in master, this is now ready for merge (kinda).

View YAML settings