Skip to content

Comments

docs(bindings/nodejs): fix broken links#6017

Merged
Xuanwo merged 6 commits intoapache:mainfrom
miroim:fix-nodejs-docs
Apr 14, 2025
Merged

docs(bindings/nodejs): fix broken links#6017
Xuanwo merged 6 commits intoapache:mainfrom
miroim:fix-nodejs-docs

Conversation

@miroim
Copy link
Member

@miroim miroim commented Apr 14, 2025

Which issue does this PR close?

Rationale for this change

before after

What changes are included in this PR?

  • Updated typedoc version from ^0.25 to ^0.28
  • Improve typedoc config to include the missing docs
  • Refactor FooterWithASFCopyright

Are there any user-facing changes?

@miroim miroim requested a review from suyanhanx as a code owner April 14, 2025 05:15
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bindings/nodejs documentation Improvements or additions to documentation labels Apr 14, 2025
@miroim miroim marked this pull request as draft April 14, 2025 07:17
@miroim miroim marked this pull request as ready for review April 14, 2025 09:02
@dosubot dosubot bot added the releases-note/docs The PR modifies docs related content or has a title that begins with "docs" label Apr 14, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you for the help!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 14, 2025
@Xuanwo Xuanwo merged commit 0da2ed4 into apache:main Apr 14, 2025
64 checks passed
@miroim miroim deleted the fix-nodejs-docs branch April 14, 2025 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bindings/nodejs documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer releases-note/docs The PR modifies docs related content or has a title that begins with "docs" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants