Skip to content

docs: add TSDoc comment to GitHubReviewers interface - #1520

Merged
orta merged 1 commit into
danger:mainfrom
Socialpranker:docs/document-github-reviewers-interface
Jul 27, 2026
Merged

docs: add TSDoc comment to GitHubReviewers interface#1520
orta merged 1 commit into
danger:mainfrom
Socialpranker:docs/document-github-reviewers-interface

Conversation

@Socialpranker

Copy link
Copy Markdown
Contributor

Summary

Closes #1409.

GitHubReviewers (referenced from GitHubJSONDSL.requested_reviewers) had no top-level TSDoc comment, so it was silently dropped from the generated DSL Reference page — typedoc only emits a dedicated section for exported interfaces/types that have a leading doc comment. Other documented interfaces in the same file (e.g. GitHubUser) already follow this pattern.

Change

Added a one-line TSDoc block above export interface GitHubReviewers in source/dsl/GitHubDSL.ts, mirroring the style used for GitHubUser.

Test plan

  • npx tsc --noEmit passes
  • npx eslint source/dsl/GitHubDSL.ts passes with no warnings
  • Comment-only change, no behavior/type change

Closes #1409. The DSL reference doc generator (typedoc) only emits a
dedicated section for exported interfaces that have a leading TSDoc
comment. GitHubReviewers lacked one, so it was silently dropped from
the generated reference even though it's referenced from
GitHubJSONDSL.requested_reviewers.
@Socialpranker

Copy link
Copy Markdown
Contributor Author

Gentle nudge on this one — approved by @fbartho on 8 July, CI green, no conflicts. It's a 3-line doc comment on GitHubReviewers, so nothing risky to land.

@orta, you've been handling merges here (#1522 went in on the 15th) — could this get picked up when you next do a pass? No rush if it's waiting on a release cycle.

@orta
orta merged commit 676d44d into danger:main Jul 27, 2026
1 check passed
@orta

orta commented Jul 27, 2026

Copy link
Copy Markdown
Member

Yeah, seems fine to me 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG][Spec] GitHubReviewers interface is not described

3 participants