doc: NAPI napi_open_callback_scope

[DaAitch]

• parameter resource_object is mandatory

Checklist

• documentation is changed or added
• commit message follows commit guidelines

Description

According to documentionnapi_open_callback_scope's parameter resource_object is optional.

[in] resource_object: An optional object associated with the async work that will be passed to possible async_hooks init hooks. 

, but when I do this:

status = napi_open_callback_scope(env, nullptr, async_context, &scope);

I get napi_invalid_arg and when I do this:

status = napi_open_callback_scope(env, resource_object, async_context, &scope);

it's napi_ok, so I checked impl and found this in node_api.cc:

v8::Local<v8::Object> resource; CHECK_TO_OBJECT(env, context, resource, resource_object); 

which makes me think that documentation is wrong, so here the PR for documentation.

[mhdawson]

LGTM since it documents current behavior.

[vsemozhetbyt]

CI-lite: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/2241/

[vsemozhetbyt]

Landed in 9e7d4ca
Thank you!