Discrepancies between code documentation online and the NPM package

[pooventhan]

Hi there!
I noticed some discrepancies with the code documentation online and the NPM package.
The web documentation says waitForVisible method returns a promise and the source file is an async method.
However, the types.d.ts file (from the NPM package) says it is a void method.

Is it a bug in .d.ts file ?

Details

• CodeceptJS version: 3.2.3
• NodeJS Version: v16.13.1
• Operating System: Windows 10

[kobenguyent]

I guess so @pooventhan as this method doesn't return anything.

[fwouts]

This appears to be caused by various mustache templates having the wrong return type, i.e. void instead of Promise<void> such as:

CodeceptJS/docs/webapi/amOnPage.mustache

Line 11 in f598059

@return {void} automatically synchronized promise with recorder #!

There are also dozens of mustache templates where a promise return type is documented in English, but not in JSDoc, such as:

CodeceptJS/docs/webapi/scrollTo.mustache

Line 12 in f598059

⚠️ returns a _promise_ which is synchronized internally by recorder

Would you like me to prepare a PR to fix this?

[kobenguyent]

This appears to be caused by various mustache templates having the wrong return type, i.e. void instead of Promise<void> such as:

CodeceptJS/docs/webapi/amOnPage.mustache

Line 11 in f598059

@return {void} automatically synchronized promise with recorder #!

There are also dozens of mustache templates where a promise return type is documented in English, but not in JSDoc, such as:

CodeceptJS/docs/webapi/scrollTo.mustache

Line 12 in f598059

⚠️ returns a _promise_ which is synchronized internally by recorder

Would you like me to prepare a PR to fix this?

That'll be great @fwouts