Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

assets: additional asset for v11.x #1673

Merged
merged 1 commit into from Feb 28, 2019

Conversation

MylesBorins
Copy link
Member

@refack
Copy link
Contributor

refack commented Jan 29, 2019

Is it needed for v10.x as well (Ref nodejs/node#22716)?

@MylesBorins
Copy link
Member Author

@refack I just took a peak at the assets for the release that needs to be promoted today and not seeing that file.

@targos
Copy link
Member

targos commented Jan 29, 2019

From the description in nodejs/node#25019, it looks like it wasn't intended to make this part of the published files?

@MylesBorins
Copy link
Member Author

hmmmm it has continued to be in the built assets so perhaps we need to revisit this?

/cc @joyeecheung

@refack
Copy link
Contributor

refack commented Jan 30, 2019

From the description in nodejs/node#25019, it looks like it wasn't intended to make this part of the published files?

AFAIK they are supposed to be published (nodejs/node#22405). They are what enable the [src] links in the docs, e.g.
image
which point to https://github.com/nodejs/node/blob/11ee7946b6b992a7dc8927cb38933475135f36a3/lib/fs.js#L172

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

semi rubber stamp LGTM, seems reasonable that we would need them and @refack has confirmed that we do.

@rvagg rvagg merged commit 6098487 into nodejs:master Feb 28, 2019
richardlau added a commit to richardlau/build that referenced this pull request Apr 9, 2020
`docs/apilinks.json` was added in v10.15.3.

Refs: nodejs/node#25019
Refs: nodejs#1673

Signed-off-by: Richard Lau <riclau@uk.ibm.com>
rvagg pushed a commit that referenced this pull request Apr 13, 2020
`docs/apilinks.json` was added in v10.15.3.

Refs: nodejs/node#25019
Refs: #1673

Signed-off-by: Richard Lau <riclau@uk.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants