Skip to content

Conversation

@avivkeller
Copy link
Member

Fixes #343.

Replaces all non-link instances of api-doc-tooling with @nodejs/doc-kit (or just doc-kit, when applicable)

Copilot AI review requested due to automatic review settings July 17, 2025 16:44
@avivkeller avivkeller requested a review from a team as a code owner July 17, 2025 16:44
@vercel
Copy link

vercel bot commented Jul 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
api-docs-tooling ✅ Ready (Inspect) Visit Preview Jul 17, 2025 4:46pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements a package renaming from api-docs-tooling to @nodejs/doc-kit to address issue #343. The changes update package metadata, documentation, and generated file comments to reflect the new naming convention.

  • Updates package name and binary command references across all documentation
  • Changes package.json to reflect new scoped package name and CLI command
  • Updates generated file comments to reference the new tool name

Reviewed Changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated no comments.

File Description
package.json Updates package name to @nodejs/doc-kit and binary command to doc-kit
README.md Updates all CLI usage examples and tool references to use new naming
CONTRIBUTING.md Updates project name references in contribution guide
src/generators/man-page/template.1 Updates generated file comment to reference new tool name

@codecov
Copy link

codecov bot commented Jul 17, 2025

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 70.80%. Comparing base (8e48f0c) to head (145fe12).

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
bin/cli.mjs 0.00% 1 Missing ⚠️
bin/commands/interactive.mjs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #358   +/-   ##
=======================================
  Coverage   70.80%   70.80%           
=======================================
  Files         128      128           
  Lines       11621    11621           
  Branches      694      694           
=======================================
  Hits         8228     8228           
  Misses       3390     3390           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ovflowd
Copy link
Member

ovflowd commented Jul 17, 2025

Fast-tracking!

@ovflowd ovflowd added the fast track This PR can land before the typical review time, with a :+1: from collaborators label Jul 17, 2025
@ovflowd ovflowd merged commit 0b65acc into main Jul 17, 2025
18 checks passed
@ovflowd ovflowd deleted the feat/name branch July 17, 2025 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast track This PR can land before the typical review time, with a :+1: from collaborators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Name the Tooling

4 participants