typedoc2md / typedoc-plugin-markdown

A plugin for TypeDoc that enables TypeScript API documentation to be generated in Markdown.
https://typedoc-plugin-markdown.org
MIT License
714 stars 176 forks source link

Potential bug with optional function argument #396

Closed Zamiell closed 5 months ago

Zamiell commented 1 year ago

Hello, I think I found a bug in the plugin.

Here's the source code for a function called "spawnCustomGridEntity": https://github.com/IsaacScript/isaacscript/blob/4d4cfd0/packages/isaacscript-common/src/classes/features/callbackLogic/CustomGridEntities.ts#L229-L237

We can see that it has, in order:

TypeDoc makes the output of the function look like this: https://isaacscript.github.io/isaacscript-common/features/CustomGridEntities#parameters

We can see that the final two arguments do not have the question mark like the other optional arguments do, which seems to be a bug.

tgreyuk commented 1 year ago

Interesting - we can easily flag default params as optional (which technically they are), but what if the first argument has a default value.? In this case you would have to pass in null or undefined to the first argument.

Zamiell commented 1 year ago

That's true. So perhaps the plugin should only mark arguments as being "optional" if they are not followed by a mandatory argument.

tgreyuk commented 5 months ago

typedoc-plugin-markdown@4.0.0