-
Notifications
You must be signed in to change notification settings - Fork 592
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
Generate docs and host on googlecloudplatform.github.io/gcloud-node ? #19
Comments
My personal choice is dox, it works nicely with jsdoc and allows examples to be documented inline. The downside is that it doesn't generate pages, but provides JSON with extracted meta information. |
@aliafshar is working with another person on the gcloud homepage. Once they come up with an initial template, we can generate Dox docs. |
@rakyll can you catch this issue up with any developments? I'm happy to start going through the code and adding examples, as well as spreading the readme out to new .md files. |
@stephenplusplus I have a meeting today at 14 Pacific to discuss about templates etc. |
Hope the meeting went well! I played around with JSDoc today - here's an example of Dataset, the generated Markdown, and its source. |
SGTM. |
@stephenplusplus could you put the PubSub refactoring on ice for the moment and work on docs for Datastore and Storage? we need to focus on completing M1, and PubSub is not part of it. Thanks! |
I would be happy to! I believe I linked to an example of a js doc overhaul Something worth noticing, which I'm sure I don't need to point out, is the As an example, if I change the Query object and update the examples in its Hope that made sense. The only alternative I can think of to address the Just things to keep in mind! On Wednesday, August 13, 2014, Silvano Luciani notifications@github.com
|
The PoC looks good to me, and in the end, as long as we're adding jsdoc, it will be easy to switch to Dox or similar. Regarding the maintenance problem, I think themaintaining samples not directly related to an API change would also apply to several .md files containing the docs. At least in this way we make sure that we can update the example directly related to a change, because it will sit in the file that we're changing :). Makes sense? |
Sure. Just to clarify the splitting of files, I meant to suggest one On Wednesday, August 13, 2014, Silvano Luciani notifications@github.com
|
Closing this since we have this launched. I'm sure we'll have tweaks as we go, but since we're live, I think we can take them on one by one. Feel free to re-open if you feel there's more to discuss. |
Sorry for being a pedant, but we're still waiting for the correct templates, I'll consider this launched when those are in place :) |
🤖 I have created a release \*beep\* \*boop\* --- ### [1.1.1](https://www.github.com/googleapis/nodejs-service-usage/compare/v1.1.0...v1.1.1) (2021-06-30) ### Bug Fixes * **deps:** google-gax v2.17.0 with mTLS ([#27](https://www.github.com/googleapis/nodejs-service-usage/issues/27)) ([c1738e8](https://www.github.com/googleapis/nodejs-service-usage/commit/c1738e82d175c921178a3d52b75197fbd791df02)) * make request optional in all cases ([#19](https://www.github.com/googleapis/nodejs-service-usage/issues/19)) ([fcdb1a5](https://www.github.com/googleapis/nodejs-service-usage/commit/fcdb1a5900b7bac5ae1b8b0e211d024b2b7e9579)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
* chore(deps): upgrade gapic-generator-java to 2.4.1 PiperOrigin-RevId: 422607515 Source-Link: googleapis/googleapis@ba2ffd6 Source-Link: googleapis/googleapis-gen@73ba4ad Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNzNiYTRhZGQyMzlhNjE5ZGE1NjdmZmJkNGU1NzMwZmRkNmRlMDRkMyJ9 * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
- [ ] Regenerate this pull request now. PiperOrigin-RevId: 474338479 Source-Link: googleapis/googleapis@d5d35e0 Source-Link: googleapis/googleapis-gen@efcd3f9 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZWZjZDNmOTM5NjJhMTAzZjY4ZjAwM2UyYTFlZWNkZTZmYTIxNmEyNyJ9
PiperOrigin-RevId: 362943541 Source-Author: Google APIs <noreply@google.com> Source-Date: Mon Mar 15 08:17:12 2021 -0700 Source-Repo: googleapis/googleapis Source-Sha: cb631dd3ffe0d6f77f7b01c5168e357771c74b51 Source-Link: googleapis/googleapis@cb631dd
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
* add try/catch to delete operations in testing * refactor test file; use timestamp for resource names, clean up resources over an hour old, move create and delete operations to before and after hooks in each test suite * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
- [ ] Regenerate this pull request now. PiperOrigin-RevId: 468790263 Source-Link: googleapis/googleapis@873ab45 Source-Link: googleapis/googleapis-gen@cb6f37a Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiY2I2ZjM3YWVmZjJhMzQ3MmU0MGE3YmJhY2U4YzY3ZDc1ZTI0YmVlNSJ9
- [ ] Regenerate this pull request now. PiperOrigin-RevId: 474338479 Source-Link: googleapis/googleapis@d5d35e0 Source-Link: googleapis/googleapis-gen@efcd3f9 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZWZjZDNmOTM5NjJhMTAzZjY4ZjAwM2UyYTFlZWNkZTZmYTIxNmEyNyJ9
🤖 I have created a release *beep* *boop* --- ## [0.2.0](googleapis/nodejs-dataform@v0.1.0...v0.2.0) (2022-09-14) ### Features * Release API version v1beta1 (no changes to v1alpha2) ([#11](googleapis/nodejs-dataform#11)) ([1ff35a6](googleapis/nodejs-dataform@1ff35a6)) ### Bug Fixes * Allow passing gax instance to client constructor ([#18](googleapis/nodejs-dataform#18)) ([34e62f5](googleapis/nodejs-dataform@34e62f5)) * Better support for fallback mode ([#13](googleapis/nodejs-dataform#13)) ([f48d918](googleapis/nodejs-dataform@f48d918)) * Change import long to require ([#14](googleapis/nodejs-dataform#14)) ([3790bfd](googleapis/nodejs-dataform@3790bfd)) * Do not import the whole google-gax from proto JS ([#1553](https://github.com/googleapis/nodejs-dataform/issues/1553)) ([#17](googleapis/nodejs-dataform#17)) ([32373fd](googleapis/nodejs-dataform@32373fd)) * Preserve default values in x-goog-request-params header ([#19](googleapis/nodejs-dataform#19)) ([bbb3790](googleapis/nodejs-dataform@bbb3790)) * Remove pip install statements ([#1546](https://github.com/googleapis/nodejs-dataform/issues/1546)) ([#15](googleapis/nodejs-dataform#15)) ([71f0add](googleapis/nodejs-dataform@71f0add)) * use google-gax v3.3.0 ([32373fd](googleapis/nodejs-dataform@32373fd)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Some options I've seen:
I personally really like the docs for Express: http://expressjs.com/3x/api.html
The text was updated successfully, but these errors were encountered: