-
Notifications
You must be signed in to change notification settings - Fork 1k
Conversation
@@ -2,20 +2,20 @@ | |||
title: 'addons' | |||
displayTitle: 'C++ addons' | |||
category: 'api' | |||
editPage: 'https://github.com/nodejs/node/blob/v18.8.0/doc/api/addons.md' | |||
editPage: 'https://github.com/nodejs/node/blob/v18.9.0/doc/api/addons.md' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside, this could probable be guess/calculated at build time to reduce git noise
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also missing the i18n part of the file path right now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also missing the i18n part of the file path right now
WDYM? The source has no i18n. The i18n is done here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside, this could probable be guess/calculated at build time to reduce git noise
Sure, but this PR is not the right place (IMHO) to talk about this. I'd recommend opening an issue with all your suggestions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, I see it's on the node repo. The URL doesn't actually take you to an editable page though, because you can't edit files on a tag path
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I understand the point of this PR now. Let's merge this ASAP as it's just a dry-run
Description
Just run
npm run sync-api
and doesn't make any other changesRelated Issues
Check List
npm run lint:js -- --fix
and/ornpm run lint:md -- --fix
for my JavaScript and/or Markdown changes.npm run test
to check if all tests are passing, and/ornpm run test -- -u
to update snapshots if I created and/or updated React Components.npm run build
work fine.