Comments in Flint configs about flintmc HOT 7 OPEN

Alternatives:

• Use a Json parser which supports comments
• Use a Json derivative which supports comments

I don't really like they idea of having configurations in multiple formats for different packages, so if we decide to use YAML or TOML I would like to not use JSON anymore.

from flintmc.

I think giving users the possibility to use their preferred format via annotation would be a nice thing. As the files are not directly accessed by other packages etc, I don’t see a problem there.

from flintmc.

Why do we need comments in the first place? These configuration files are not supposed to be edited manually. Default values should be provided via annotations, not via a file. Just comment in the configuration interface.

from flintmc.

I think comments generally make sense, sometimes you need to edit the configuration manually, either because your game wont start anymore or because some options are hidden from the GUI as they are debug only options.

from flintmc.

Both cases should only be necessary for developers that have access to the original model interface. Debug options should still be visible in GUI, just hiddem behind a "Show Debug Options" setting or something like that. In my opinion it should be considered an implementation detail how and where we store serialized configurations. We might as well use a SQLite databse...
If the game doesn't start because of a configuration issue then that is clearly a bug in the package that is causing the crash. We could consider a "Safe Mode Startup" that automatically uses default configurations for all configs.

Anyways, if you wanna add this feature, go ahead, I'm fine with it, though I really think it is unnecessary and we shouldn't support manual config editing in this way.

from flintmc.

I'm willing to reconsider my opinion on this once (or should I say if) we support serverside modding.

from flintmc.

Yeah, that’s a point where this can be useful. In general, the config system is not depending on Minecraft, so it might be used in other contexts where comments and editing the config directly can be useful.

from flintmc.