deephaven / web-client-ui

Deephaven Web Client UI
Apache License 2.0
29 stars 31 forks source link

fix: Change ruff errors to warnings and fix config saving #2246

Closed mattrunyon closed 1 month ago

mattrunyon commented 1 month ago

Changed all except syntax errors to warning underlines instead of error underline since they are not actual errors.

Also changed redux to not save if the config matches the default. This should fix an edge case where a user opens the settings editor just to see what the config is, then hits "Save". Previously, this would save the current defaults as their settings and then if the defaults changed they would not get updated settings. This way, if a user was on defaults and the default changes they will get the new default.

Also added a little bit of debug logging for Ruff.