NeosModLoader icon indicating copy to clipboard operation
NeosModLoader copied to clipboard

Implement #75 - Command Line Argument parsing utility and config

Open Banane9 opened this issue 2 years ago • 2 comments

Implements an expanded upon Version of the feature requested by @art0007i in #75.

Also changed config loading to use reflection to make it more compact, more permissive and easy to add new values.
Added some inverted properties to allow setting them with flag arguments.

Banane9 avatar Mar 30 '23 21:03 Banane9

maybe instead of putting all the neos default argument parsing code inside of the loader itself, it would be better to create a library mod that can be used by other mods? we have Libraries category in the manifest that seems like it would be perfect for this

art0007i avatar Mar 31 '23 14:03 art0007i

I'm generally in favor of making things more generic like this would do with parsing the config,

hey, doing the config parsing "manually" is @zkxs's doing. I've used an ini Parser in the SpotifyStatus server, but that's another library and also only parses it into a string dictionary style afaik

maybe instead of putting all the neos default argument parsing code inside of the loader itself, it would be better to create a library mod that can be used by other mods? we have Libraries category in the manifest that seems like it would be perfect for this

but if it's a library I can't use it for NML's config 😝

Banane9 avatar Mar 31 '23 16:03 Banane9