there is no sense in me keeping it around when all it does is
add faff to writing the script
add useless noise to the filenames/paths
and enforces its opinions upon me which i dont agree with
and all for absolutely ZERO reason since i don't even have a feed in the
first place
if you want to take my scripts and package them into a dependencycontrol
feed, you are more than welcome to do so, provided you comply with the
terms of the licence. but your life is going to be a little bit harder
i'm afraid, sorry about that.
i will keep all the script_namespace, depctrl registration and such
around for the time being (removing it is >effort)
not as thorough as `do` because most of `undo` is pretty clear
on its own, and all of the "magic" bits have been explained in `do`.
The only difference is that here it's the other way round.
i find it easier to read when the different parts are separated,
and while i have done one-line `if this then do that end`s before,
i've only ever done it for short operations where it's immediately
obvious what's going on and you don't need to do any thinking whatsoever.
e.g. this function from depctrl config:
local function get_log_level(num)
if num == 0 then return "0: Fatal"
elseif num == 1 then return "1: Error"
elseif num == 2 then return "2: Warning"
elseif num == 3 then return "3: Hint"
elseif num == 4 then return "4: Debug"
elseif num == 5 then return "5: Trace" end
return nil
end
I don't think the bits that fix sel and act count, at least not without comments.
speaking of comments, i've added some.
i hope i've understood what's going on correctly, please feel free to correct me if not.
local functions
split a thing off into a function
sane function name
hmm lot of functions this time
also local variables and addressing luacheck warnings
better than finding a file that's probably the config, parsing the json, and writing it back in,
for obvious reasons
turns out no i wasnt thinning out the soup too much
twas just right
Reason for adding values one by one in key-pair loop instead of just config = new_config:
> [01:31] arch1t3cht: oh, I think you can't assign entire tables to DependencyControl.config.c
> [01:31] arch1t3cht: like, you can't do DependencyControl.config.c = <my new config>,
> you should do DependencyControl.config.myfield = <my new value> and whatnot
> [01:32] arch1t3cht: because behind the scenes it uses metatable magic to track what field changes
> so it can merge its own changes with ones that other scripts made in the meantime
Co-Authored-By: arch1t3cht <arch1t3cht@gmail.com>
