Team-Silver-Sphere / SquadJS

Squad Server Script Framework
Boost Software License 1.0
167 stars 128 forks source link

Multi-config using config includes #320

Closed steelskillet closed 3 months ago

steelskillet commented 1 year ago

Based on changes in #283. Provides specifying config files to include before and after a config file to append to a config list. It then applies the values from each json config in the list in order overwriting previous values. includes are recursive and errors on circular dependency.

This allows easily sharing a bundle of configs and puts into the actual configuration, what other configs are required for a specific config file.

needs extended testing (changes will be merged into ASG servers), and a double check of the changes to the documentation at the moment (and maybe a little clean up of the code).

steelskillet commented 1 year ago

small test with ASG servers, seems to be working as expected. Worked with no change to original config file, both includes and overwrites seems to be working as expected as well.

I would like to get a better log entry for printing the config out, one that lists the absolute path of the file that put the value in for each config entry to help out with debugging what configs are setting what value. This will require some non-trivial changes however and might take a bit.

werewolfboy13 commented 10 months ago

Needs rewrite

steelskillet commented 10 months ago

if it is just the conflicts, we good now. if there is anything specific then list would be good.

steelskillet commented 6 months ago

This doesn't work how i imagined it in the current state, i plan on working on it fairly soon (tm) so i will leave this open for now, just expect large changes or this getting closed and a new one reopened.