[FEATURE] Removed unused TOML variables
Opened this issue · 0 comments
Description
Through my work in #871 it was not always clear if any of the variables actually served a purpose. I found myself doing a lot of work to ensure that most of the variables were assigned at a bare minimum. Even if they are assigned, it is unclear if they are ever referenced elsewhere. This can cause a frustrating user experience, where they change a seemingly useful variable only to find it has no effect.
I could potentially make the changes myself given my work in #871, so I would be happy to work on that if needed.
Describe the solution you'd like
Remove all TOML variables that are never used in any meaningful manner (i.e. remove anything that is never used after assignment).
Describe any alternative solutions/features you've considered
We could leave a comment/disclaimer that the variable is not going to be used. This could be particularly good for if we plan to implement/reroute some variables in the future, or if they are required in function headers. However, I would prefer to delete over leaving a comment whenever possible.
Any additional context or information about the feature request
Any changes made to the file should also be reflected in the example TOML, either by a comment stating the variable does nothing, or removing it entirely.
It's entirely possible this issue gets closed with no changes made, and that's fine as long as we can ensure most, if not all of the TOML variables can be checked to ensure they actually make changes.