Skip to content

Commit

Permalink
fix(cli/napi): property with getter/setter always failed (denoland#19562
Browse files Browse the repository at this point in the history
)

with `napi_define_properties`.
  • Loading branch information
DjDeveloperr authored Jun 21, 2023
1 parent 6729e9c commit c9e4780
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cli/napi/js_native_api.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1559,7 +1559,7 @@ fn napi_define_properties(
let define_maybe = object.define_property(scope, name, &desc);
return_status_if_false!(
env_ptr,
!define_maybe.unwrap_or(false),
define_maybe.is_some(),
napi_invalid_arg
);
} else if property.method.is_some() {
Expand Down

0 comments on commit c9e4780

Please sign in to comment.