dan/oxc
1
0
Fork 0
mirror of https://github.com/danbulant/oxc synced 2026-05-19 04:08:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(linter): Correct configuration file parsing for jsx-no-useless-fragment (#2512)

Browse source 
fix: #2484 

I fixed the configuration parsing for jsx-no-useless-fragment.
This commit is contained in:
keita hino 2024-02-26 16:49:50 +09:00 committed by GitHub
parent cd75c1ca59
commit 93742f89e9
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 9 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
crates/oxc_linter/src/rules/react/jsx_no_useless_fragment.rs
View file

@ -56,10 +56,13 @@ declare_oxc_lint!(
impl Rule for JsxNoUselessFragment {
fn from_configuration(value: serde_json::Value) -> Self {
let allow_expressions =
value.get("allowExpressions").and_then(serde_json::Value::as_bool).unwrap_or(false);
let value = value.as_array().and_then(|arr| arr.first()).and_then(|val| val.as_object());
Self { allow_expressions }
Self {
allow_expressions: value
.and_then(|val| val.get("allowExpressions").and_then(serde_json::Value::as_bool))
.unwrap_or(true),
}
}
fn run<'a>(&self, node: &AstNode<'a>, ctx: &LintContext<'a>) {
@ -224,15 +227,16 @@ fn test() {
(r"<Fragment key={item.id}>{item.value}</Fragment>", None),
(r"<Fooo content={<>eeee ee eeeeeee eeeeeeee</>} />", None),
(r"<>{foos.map(foo => foo)}</>", None),
(r"<>{moo}</>", Some(json!({ "allowExpressions": true }))),
(r"<>{moo}</>", Some(json!([{ "allowExpressions": true }]))),
(
r"
<>
{moo}
</>
",
Some(json!({ "allowExpressions": true })),
Some(json!([{ "allowExpressions": true }])),
),
(r"{1 && <>{1}</>}", Some(json!(["warn", {"allowExpressions": true}]))),
];
let fail = vec![