Skip to content

fix(docs-infra): strip newlines from class signature in API gen#69606

Open
aparzi wants to merge 1 commit into
angular:mainfrom
aparzi:fix-reference-doc-api-links
Open

fix(docs-infra): strip newlines from class signature in API gen#69606
aparzi wants to merge 1 commit into
angular:mainfrom
aparzi:fix-reference-doc-api-links

Conversation

@aparzi

@aparzi aparzi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Prevents multiline generics or implements clauses from breaking the rendered class signature in the API reference docs.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

Issue Number: #58053

Does this PR introduce a breaking change?

  • Yes
  • No

Prevents multiline generics or implements clauses from breaking the
rendered class signature in the API reference docs.
@pullapprove pullapprove Bot requested a review from josephperrott July 1, 2026 15:51
@angular-robot angular-robot Bot added the area: docs-infra Angular.dev application and infrastructure label Jul 1, 2026
@ngbot ngbot Bot added this to the Backlog milestone Jul 1, 2026
@JeanMeche JeanMeche requested review from JeanMeche and removed request for josephperrott July 1, 2026 15:56
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deployed adev-preview for 80d3786 to: https://ng-dev-previews-fw--pr-angular-angular-69606-adev-prev-c8n6v71z.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@JeanMeche JeanMeche left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm surprised it was that simple 😄

@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jul 1, 2026
@aparzi

aparzi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

I'm surprised it was that simple 😄

Haha right? Best kind of PRs! Glad it turned out that way

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

Labels

action: merge The PR is ready for merge by the caretaker adev: preview area: docs-infra Angular.dev application and infrastructure target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants