thetawavegame / thetawave

A physics based, space shooter game made with Rust and the Bevy engine.
https://thetawave.metalmancy.tech
MIT License
164 stars 9 forks source link

Replace Options Config with CLI Parameters #112

Open cdsupina opened 1 year ago

cdsupina commented 1 year ago
          I think we should delete this. CLI parameters are a more standard way to do this. `thetawave --resolution 1920x1080 --fullscreen` and those can be used for launch options in steam. I dont think we want others to change this via ron (99.99% of people dont know what RON is, but almost all programmers can do `thetawave --help` and set CLI params). 

https://github.com/thetawavegame/thetawave/blob/0447952b94077c66cb8bc92fcaa4e210ab307aff/src/options/mod.rs#L26

Atleast outside of the browser, we can save user settings in sqlite. Keeping a couple of preconfigured ones as ron is...fine for now (if they can be toml and are simple enough, I think toml is better and more portable/standard; i dont particularly value serde's data model enough to have a strong preference for ron).

_Originally posted by @varoonp123 in https://github.com/thetawavegame/thetawave/pull/108#discussion_r1321732859_