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

v12.x LTS expected assets need updating #2276

Closed
codebytere opened this issue Apr 8, 2020 · 6 comments · Fixed by nodejs/node#32728
Closed

v12.x LTS expected assets need updating #2276

codebytere opened this issue Apr 8, 2020 · 6 comments · Fixed by nodejs/node#32728
Assignees

Comments

@codebytere
Copy link
Member

Discovered during the latest v12.16.2 release:

# Promoting v12.16.2...
... Checking assets
... Expecting a total of 39 assets for v12.x
... 40 assets waiting in staging
... 0 assets already promoted
 ✓  Complete set of expected assets in place for v12.x
 ✖  The following assets were found in staging but are not expected for v12.x:
    • docs/previous-versions.json
    Does the expected assets list for v12.x need to be updated?
    https://github.com/nodejs/build/tree/master/ansible/www-standalone/tools/promote/expected_assets/v12.x

I think that just means adding the new asset here?

@targos
Copy link
Member

targos commented Apr 8, 2020

This was added in nodejs/node#32518

@richardlau is it expected to be part of the published assets?

@richardlau
Copy link
Member

This was added in nodejs/node#32518

@richardlau is it expected to be part of the published assets?

Didn’t expect that. The file doesn’t need to be there as it’s only used in building the docs and not referenced by the built docs.

@codebytere
Copy link
Member Author

Hmm - is there a solution moving forward that's not just adding it to expected assets?

@richardlau
Copy link
Member

Hmm - is there a solution moving forward that's not just adding it to expected assets?

nodejs/node#32728 moves (and renames) the file so that it is outside of out/doc which should prevent it being published.

@codebytere
Copy link
Member Author

Thanks @richardlau - I'll close my PR!

@rvagg
Copy link
Member

rvagg commented Apr 9, 2020

can I just say, yay 🍾 for expected-assets, working really nicely if it's even picking up things like this.

@rvagg rvagg closed this as completed Apr 9, 2020
richardlau added a commit to richardlau/node-1 that referenced this issue Apr 11, 2020
Move the generated previous doc versions JSON file out of `out/doc` to
prevent it being included in the distributed packages.

Signed-off-by: Richard Lau <riclau@uk.ibm.com>

PR-URL: nodejs#32728
Fixes: nodejs/build#2276
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
targos pushed a commit to nodejs/node that referenced this issue Apr 13, 2020
Move the generated previous doc versions JSON file out of `out/doc` to
prevent it being included in the distributed packages.

Signed-off-by: Richard Lau <riclau@uk.ibm.com>

PR-URL: #32728
Fixes: nodejs/build#2276
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
BethGriggs pushed a commit to nodejs/node that referenced this issue Apr 14, 2020
Move the generated previous doc versions JSON file out of `out/doc` to
prevent it being included in the distributed packages.

Signed-off-by: Richard Lau <riclau@uk.ibm.com>

PR-URL: #32728
Fixes: nodejs/build#2276
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
richardlau added a commit to nodejs/node that referenced this issue Apr 17, 2020
Move the generated previous doc versions JSON file out of `out/doc` to
prevent it being included in the distributed packages.

Signed-off-by: Richard Lau <riclau@uk.ibm.com>

PR-URL: #32728
Fixes: nodejs/build#2276
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
targos pushed a commit to nodejs/node that referenced this issue Apr 22, 2020
Move the generated previous doc versions JSON file out of `out/doc` to
prevent it being included in the distributed packages.

Signed-off-by: Richard Lau <riclau@uk.ibm.com>

PR-URL: #32728
Fixes: nodejs/build#2276
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
BethGriggs pushed a commit to nodejs/node that referenced this issue Jun 2, 2020
Move the generated previous doc versions JSON file out of `out/doc` to
prevent it being included in the distributed packages.

Signed-off-by: Richard Lau <riclau@uk.ibm.com>

PR-URL: #32728
Fixes: nodejs/build#2276
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants