Skip to content

Commit

Permalink
theme: Include key names in style parsing warnings
Browse files Browse the repository at this point in the history
This should make it easier to figure out why the theme-check CI job
fails. Previously the message didn't include the failing key so you
were left searching or guessing where the error occurred.
  • Loading branch information
the-mikedavis committed Dec 4, 2024
1 parent 7a2afdc commit 07e7e75
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion helix-view/src/theme.rs
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ fn build_theme_values(
for (name, style_value) in values {
let mut style = Style::default();
if let Err(err) = palette.parse_style(&mut style, style_value) {
warnings.push(err);
warnings.push(format!("Failed to parse style for key {name:?}. {err}"));
}

// these are used both as UI and as highlights
Expand Down

0 comments on commit 07e7e75

Please sign in to comment.