From a4a5d101707efac693f7fc0f82e312e65ee1912e Mon Sep 17 00:00:00 2001 From: isnaini <68492618+isnakode@users.noreply.github.com> Date: Sat, 20 Apr 2024 13:34:14 +0800 Subject: [PATCH] Apply suggestions from code review Co-authored-by: ty <62130798+togami2864@users.noreply.github.com> --- .../lint/nursery/no_duplicate_selectors_keyframe_block.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/crates/biome_css_analyze/src/lint/nursery/no_duplicate_selectors_keyframe_block.rs b/crates/biome_css_analyze/src/lint/nursery/no_duplicate_selectors_keyframe_block.rs index 4a3f66d815b6..8f7c70a8e620 100644 --- a/crates/biome_css_analyze/src/lint/nursery/no_duplicate_selectors_keyframe_block.rs +++ b/crates/biome_css_analyze/src/lint/nursery/no_duplicate_selectors_keyframe_block.rs @@ -54,8 +54,7 @@ impl Rule for NoDuplicateSelectorsKeyframeBlock { for keyframe_item in node.items() { match keyframe_item { AnyCssKeyframesItem::CssKeyframesItem(item) => { - let keyframe_selector = item.selectors().into_iter().next()?; - let keyframe_selector = keyframe_selector.ok()?; + let keyframe_selector = item.selectors().into_iter().next()?.ok()?; if !selector_list.insert(keyframe_selector.text().to_lowercase()) { return Some(keyframe_selector); } @@ -76,7 +75,7 @@ impl Rule for NoDuplicateSelectorsKeyframeBlock { }, ) .note(markup! { - "Consider using different selector here!" + "Consider using a different percentage value or keyword to avoid duplication" }), ) }