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

tools: fix docopen target #35062

Closed
wants to merge 1 commit into from
Closed

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented Sep 5, 2020

Adds to prerequisites the output file to open. Previously, this would open an out-of-date version of the file, or fail if it has not been built before.

$(apidocs_html) is part of out/doc/api/all.html prerequisites, it's safe use out/doc/api/all.html as the only prerequisite here.

node/Makefile

Lines 766 to 768 in f3b4846

out/doc/api/all.html: $(apidocs_html) tools/doc/allhtml.js \
tools/doc/apilinks.js
$(call available-node, tools/doc/allhtml.js)

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Adds to prerequisites the output file to open. Previously, this would
open an out-of-date version of the file, or fail if it hasn't been built
before.
@nodejs-github-bot nodejs-github-bot added the build Issues and PRs related to build files or the CI. label Sep 5, 2020
@codecov-commenter
Copy link

Codecov Report

Merging #35062 into master will decrease coverage by 0.02%.
The diff coverage is 95.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #35062      +/-   ##
==========================================
- Coverage   96.69%   96.67%   -0.03%     
==========================================
  Files         207      207              
  Lines       69191    69305     +114     
==========================================
+ Hits        66902    66998      +96     
- Misses       2289     2307      +18     
Impacted Files Coverage Δ
lib/internal/crypto/random.js 97.12% <92.40%> (-2.88%) ⬇️
lib/_stream_readable.js 99.01% <100.00%> (+<0.01%) ⬆️
lib/crypto.js 96.95% <100.00%> (+0.02%) ⬆️
lib/events.js 98.82% <100.00%> (+0.03%) ⬆️
lib/internal/buffer.js 100.00% <100.00%> (ø)
lib/querystring.js 99.29% <100.00%> (ø)
lib/internal/child_process.js 95.46% <0.00%> (-1.12%) ⬇️
lib/_http_server.js 98.40% <0.00%> (-0.23%) ⬇️
lib/internal/util/inspect.js 96.12% <0.00%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 89f2d46...fa8fbd6. Read the comment docs.

@DerekNonGeneric DerekNonGeneric added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 5, 2020
DerekNonGeneric pushed a commit that referenced this pull request Sep 6, 2020
Adds to prerequisites the output file to open. Previously, this would
open an out-of-date version of the file, or fail if it hasn't been built
before.

PR-URL: #35062
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
@DerekNonGeneric
Copy link
Contributor

Landed in 441c03d

@aduh95 aduh95 deleted the fix-make-docopen-target branch September 6, 2020 20:39
richardlau pushed a commit that referenced this pull request Sep 7, 2020
Adds to prerequisites the output file to open. Previously, this would
open an out-of-date version of the file, or fail if it hasn't been built
before.

PR-URL: #35062
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
@richardlau richardlau mentioned this pull request Sep 7, 2020
4 tasks
richardlau pushed a commit that referenced this pull request Sep 7, 2020
Adds to prerequisites the output file to open. Previously, this would
open an out-of-date version of the file, or fail if it hasn't been built
before.

PR-URL: #35062
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
Adds to prerequisites the output file to open. Previously, this would
open an out-of-date version of the file, or fail if it hasn't been built
before.

PR-URL: #35062
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
Adds to prerequisites the output file to open. Previously, this would
open an out-of-date version of the file, or fail if it hasn't been built
before.

PR-URL: #35062
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
@codebytere codebytere mentioned this pull request Sep 28, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
Adds to prerequisites the output file to open. Previously, this would
open an out-of-date version of the file, or fail if it hasn't been built
before.

PR-URL: nodejs#35062
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Derek Lewis <DerekNonGeneric@inf.is>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants