Skip to content

fix(dgeni): const functions don't show params#4393

Open
griest024 wants to merge 1 commit intograycoreio:developfrom
griest024:fix/dgeni/const-functions
Open

fix(dgeni): const functions don't show params#4393
griest024 wants to merge 1 commit intograycoreio:developfrom
griest024:fix/dgeni/const-functions

Conversation

@griest024
Copy link
Copy Markdown
Member

PR Checklist

  • Commit message follows our contributing guidelines
  • Tests added/updated (for bug fixes/features)
  • Documentation added/updated (for bug fixes/features)

PR Type

  • Bug fix
  • Feature
  • Style update
  • Refactor
  • Test
  • Build
  • CI
  • Docs
  • Performance
  • Other (please describe)

Current behavior

Fixes: #3733
Part of: #

New behavior

Breaking change?

  • Yes
  • No

Additional context

@griest024 griest024 requested review from a team as code owners March 23, 2026 21:27
@griest024 griest024 requested a review from xelaint March 23, 2026 21:27
Copy link
Copy Markdown
Member

@xelaint xelaint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see the params on the page documented in the issue:

Screen Shot 2026-03-24 at 10 08 38 AM

Comment thread libs/design/toast/src/interfaces/toast-options.ts Outdated
@griest024 griest024 force-pushed the fix/dgeni/const-functions branch from 52c9bb7 to 85c2dc8 Compare March 26, 2026 17:31
@xelaint
Copy link
Copy Markdown
Member

xelaint commented Mar 30, 2026

I'm still not seeing the param docs even after adding test docs like this:

 * @param input - test
 * @returns - test

@xelaint xelaint added status: needs response This PR is waiting for a response from the author tools: dgeni @daffodil/tools-dgeni labels Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: needs response This PR is waiting for a response from the author tools: dgeni @daffodil/tools-dgeni

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API functions are missing argument documentation

2 participants