-
Notifications
You must be signed in to change notification settings - Fork 113
404 on tracing module (API) #77
Comments
good catch! i opened a PR to adress the problem |
@robertkowalski I don't see in this repository the PR you mention, and the problem seems to not be fixed. Could you please create it so that we can merge it and fix this issue? Thanks! |
yes @paulomcnally closed the issue still waiting for a review on nodejs/node-v0.x-archive#9172 - PTAL :) |
@misterdjules moved to: nodejs/node-v0.x-archive#9172 |
@robertkowalski Found it: nodejs/node-v0.x-archive#9172. |
@paulomcnally Thanks :) The fix landed in nodejs/node-v0.x-archive@08a1192. With this fix and following our usual workflow, the API doc's table of content would be updated when node v0.12.1 is released. I would like to deploy this fix sooner than that though, as we still don't know when this release will happen. @tjfontaine @robertkowalski It should be safe to |
I deployed nodejs/node-v0.x-archive@08a1192 to nodejs.org to avoid having to wait for v0.12.1 to fix this annoying and confusing issue. I cherry-picked nodejs/node-v0.x-archive@08a1192 and diffed the content of the generated api doc, and then ran Closing as the TOC at http://nodejs.org/api/ now does not include the tracing module anymore. Thank you @paulomcnally and @robertkowalski! |
I use this url to create local documentation https://github.com/nodenica/grunt-node-api-docs-responsive https://nodejs.org/dist/v0.12.0/docs/api/index.json tracing link exists on this source :/ |
@misterdjules did you port it to 0.12 ? |
The fix has been merged into the v0.12 branch, but not into the v0.12.0-release branch. Thus, it's been deployed to The v0.12.0 release's artifacts should in my opinion remain intact, and the fix will be present in all subsequent v0.12.x releases. @paulomcnally Is there a way for you to handle this issue for v0.12.0 in your application? |
Yes @misterdjules for this version (only for this version) i use /api/index.json and not /dist/v0.12.0/docs/api/index.json. |
@paulomcnally |
@misterdjules /api/index.json now it's for my custom app. The module use http://nodejs.org/dist/v0.12.0/docs/api/index.json This solved on next version. From now create a tracing.html empty. |
The problem is still present in http://nodejs.org/dist/v0.12.1/docs/api/ because the v0.12.1 release was branched off v0.12.0 and just added the OpenSSL upgrade, thus not including nodejs/node-v0.x-archive@08a1192. http://nodejs.org/dist/v0.12.2/docs/api/ will definitely not have this problem, sorry for the inconvenience. |
i know :/ Thanks @misterdjules |
http://nodejs.org/api/tracing.html
The text was updated successfully, but these errors were encountered: