Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

chore: run sync-api for 18.9.0 docs #2759

Merged
merged 1 commit into from
Sep 12, 2022
Merged

chore: run sync-api for 18.9.0 docs #2759

merged 1 commit into from
Sep 12, 2022

Conversation

nschonni
Copy link
Member

@nschonni nschonni commented Sep 12, 2022

Description

Just run npm run sync-api and doesn't make any other changes

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npm run lint:js -- --fix and/or npm run lint:md -- --fix for my JavaScript and/or Markdown changes.
    • This is important as most of the cases your code changes might not be correctly linted
  • I have run npm run test to check if all tests are passing, and/or npm run test -- -u to update snapshots if I created and/or updated React Components.
  • I have checked that the build works locally and that npm run build work fine.
  • I've covered new added functionality with unit tests if necessary.

@@ -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'
Copy link
Member Author

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

Copy link
Member Author

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

Copy link
Member

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.

Copy link
Member

@ovflowd ovflowd Sep 12, 2022

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.

Copy link
Member Author

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

@nschonni nschonni requested a review from ovflowd September 12, 2022 21:41
Copy link
Member

@ovflowd ovflowd left a 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

@ovflowd ovflowd merged commit a4160b8 into nodejs:main Sep 12, 2022
@nschonni nschonni deleted the 18.9.0 branch September 12, 2022 22:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants