Josee9988
commented
2 years ago Check that DeepScan error. Seems like MinifyOptions is not used. Have you modified anything that caused this error? I'll check it in a bit anyways Thanks as always @Cube707
Closed Cube707 closed 2 years ago
Josee9988
commented
2 years ago Check that DeepScan error. Seems like MinifyOptions is not used. Have you modified anything that caused this error? I'll check it in a bit anyways Thanks as always @Cube707
Cube707
commented
2 years ago Yes, I removed that and will also remove the import if thats ok with you.
I removed it to make the update process simpler (not copying the terserminifyoptions to a second variable but using the settings.terseroptions directly)
Josee9988
commented
2 years ago Seems like everything works as expected. I'll merge it, fix some minor linter problems and release a new version
Realtime config updates
this PR implements a new feature, as discussed in #132
Description
A eventhandler is now atatched to the
onDidChangeConfiguration, allowing for silent internal updating of most configurations.Some special configs still need a window reload, so the user will be promted for it.
Additional context
to simplify the updating process (and the code in generall) I removed two globel variables that just stored some parts of the settings. Now the settings interface is used directly.