-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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
doc: list the stability status of each API in one place? #23723
Comments
I can pick this one |
I expect to make the first commit of nodejs. I can gather the stability of APIs in one place. |
I suppose we may need to automate this with doc tools scripts. Otherwise, each deprecation and API addition will need to be addressed in this list manually which may be error-prone. |
@vsemozhetbyt Good idea. Can you leave a few pointers? I only have a vague idea that people who want to hack on this should start in https://github.com/nodejs/node/tree/master/tools/doc Also cc @nodejs/documentation |
Recently, we change our toolchain a bit, so maybe @rubys can give some hints. I think we have at least two options:
|
I recommend that all.json approach. The input data is already parsed. Iterate over the structure, producing output as you go. The tool should go in the |
I hope to try this work. |
@HYU-LeeHG are you actively working on this issue? If not, it would be great to take it |
@rubys I went ahead wrote a script to get all the stability keys from the json mentioned.
What are your thoughts on organizing this? What can be the next steps? |
can i take this issue |
Hi, I was looking for a good first issue and ran into this issue. So shobhitchittora@b0c5461 by @shobhitchittora already collects stability statuses and outputs them into a table. The problem is how to put it into the docs. Possible options would be:
The option 1 seems easiest to me. What do you think? @joyeecheung @rubys @shobhitchittora |
@shuhei I'd prefer we do option 2 if possible, but we could also implement option 1 first (in any cases I can imagine a |
If no one is working on it I can give it a try. |
Would this include only the top level API's (as in @shobhitchittora 's suggestion above) or do we want to breakdown the I have a suggested solution which I will move into a PR soon. My solution produces similar output to @shobhitchittora however it goes further to look for the My question, however, is where would this script live in terms of a directory? Here is an example on how the output would look like for the current latest version(
|
Hi, can I take this issue? |
Awesome, I think the right place would be here at https://nodejs.org/api/documentation.html#documentation_stability_index |
If no one is working on it |
@manisaiprasad You can give it a try. |
1 similar comment
This comment has been minimized.
This comment has been minimized.
@shobhitchittora can I try this one? |
Hey! First-time contributor here. Can I take this up if no one is working on it? |
Hi @iammack I think the best way to move forward would be to try opening a PR with the suggestions above. Adding the script to |
I agree. I think that @shobhitchittora and @martin-dimitrov13 already have branches with their solutions. If neither of you is continuing with it, I can proceed. |
I'm working on this issue. |
Fixes: #23723 PR-URL: #36223 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
Fixes: #23723 PR-URL: #36223 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
Fixes: #23723 PR-URL: #36223 Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Myles Borins <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
Right now each API has their own page with their stability index, but AFAIK there is not a central place where you can find the stability status of all the APIs at a glance (correct me if I am wrong). It may be useful to have a list somewhere in https://github.com/nodejs/node/blob/master/doc/api/documentation.md#stability-index
The text was updated successfully, but these errors were encountered: