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

[CHA-188] Added typedoc to package reqs. #107

Merged
merged 11 commits into from
Apr 30, 2024
Merged

Conversation

splindsay-92
Copy link
Contributor

Setup basic typedoc config.
Added landing page for typedoc.

Related to infra PR here to add a required role.

Screenshot 2024-04-26 at 14 58 04

"out": "typedoc/generated",
"readme": "typedoc/landing-page.md",
"tsconfig": "tsconfig.base.json",
"treatWarningsAsErrors": false,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is set to false as we have a lot of things to fix documentation-wise, but I've added all the validation in anyway so when running it locally you can just flick the switch and see what needs to be fixed.

@splindsay-92 splindsay-92 force-pushed the CHA-188-bootstrap-typedoc branch from 689e0c4 to e98d455 Compare April 26, 2024 15:00
@@ -0,0 +1,42 @@
name: API Reference
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo in the yaml file name :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry Andy, I think I'm being blind and can't see the issue? 🙈

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's called docks.yml rather than docs.yml :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You know, I think I might have taken a year to spot that, my brain simply wouldn't see it XD

.github/workflows/docks.yml Outdated Show resolved Hide resolved
tsconfig.base.json Outdated Show resolved Hide resolved
@@ -0,0 +1,6 @@
# Ably Chat JavaScript Client Library SDK API Reference

The JavaScript Client Library SDK supports chat functionality over ably realtime. The JavaScript API references are generated from the [Ably Chat JavaScript Client Library SDK source code](https://github.com/ably-labs/ably-chat-js) using [TypeDoc](https://typedoc.org) and structured by classes.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The JavaScript Client Library SDK supports chat functionality over ably realtime. The JavaScript API references are generated from the [Ably Chat JavaScript Client Library SDK source code](https://github.com/ably-labs/ably-chat-js) using [TypeDoc](https://typedoc.org) and structured by classes.
The JavaScript Client Library SDK supports chat functionality over Ably. The JavaScript API references are generated from the [Ably Chat JavaScript Client Library SDK source code](https://github.com/ably-labs/ably-chat-js) using [TypeDoc](https://typedoc.org) and structured by classes.

@AndyTWF
Copy link
Collaborator

AndyTWF commented Apr 29, 2024

Also - please add the JIRA ticket number to the issue title :)

@splindsay-92 splindsay-92 changed the title Added typedoc to package reqs. [CHA-188] Added typedoc to package reqs. Apr 29, 2024
@splindsay-92 splindsay-92 merged commit 9469190 into main Apr 30, 2024
6 checks passed
@AndyTWF AndyTWF deleted the CHA-188-bootstrap-typedoc branch July 10, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants