Use tool --fix-config to remove JSON too, log frontend activity - #175
Closed
almostearthling wants to merge 25 commits into
Closed
Use tool --fix-config to remove JSON too, log frontend activity#175almostearthling wants to merge 25 commits into
--fix-config to remove JSON too, log frontend activity#175almostearthling wants to merge 25 commits into
Conversation
The `parameter_call` entry was left out from JSON to TOML conversion when writing a configuration file, leading to possibly deprecated configurations if the DBus condition to be saved was not newly created.
Step 1 toward fixing #174
Actually load the fixed configuration items and globals, and then use the standard writer to write a human-readable configuration file even if the configuration was created by old versions of When, which used to produce hard-to-read configuration files
In order to solve #177, the logger should be generally accessible from other parts of the application than just the wrapper: now the logger is managed by the utility module, for the moment with no changes
The logging `Context` structure is made available by the logger, in order to better define the context in various parts of the application and only modify variable parts of the context when it is necessary to log a message
almostearthling
changed the title
--fix-config to remove JSON too--fix-config to remove JSON too, log frontend activity
Owner
Author
|
This also closes #177 |
Closed
Owner
Author
|
Testing on Linux succeeded as well: merging in a few days. |
Rewrite the README file in order to be more useful for its purpose, and move parts that are not README-specific to the detailed documentation
Owner
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Launching
now removes JSON based configuration fron older file, changing it into the equivalent (and compliant to the upcoming version of whenever) pure-TOML form. This is done automatically on all items that are affected.
Fixes #174