Skip to content

Set up API docs generation [SDK-2921]#587

Merged
Widcket merged 4 commits intobetafrom
v2/jazzy
Dec 14, 2021
Merged

Set up API docs generation [SDK-2921]#587
Widcket merged 4 commits intobetafrom
v2/jazzy

Conversation

@Widcket
Copy link
Copy Markdown
Contributor

@Widcket Widcket commented Dec 14, 2021

Changes

This PR sets up the API docs generation with Jazzy, and raises the documentation coverage to 92% of the public API surface area.
The command that generates the docs (bundle exec jazzy) was added to the .shiprc file to generate the docs when creating a release PR.

Screen Shot 2021-12-13 at 21 13 42

The initial generated docs will be added in a future PR.

Testing

  • This change adds unit test coverage
  • This change has been tested on the latest version of the platform/language or why not

Checklist

@Widcket Widcket requested a review from a team as a code owner December 14, 2021 00:23
@Widcket Widcket added the review:large Large review label Dec 14, 2021
- scope: Scope value requested when authenticating the user. Default is `openid profile email`.
- Returns: Authentication request that will yield Auth0 user's credentials.
- Requires: Passwordless OTP Grant `http://auth0.com/oauth/grant-type/passwordless/otp`. Check [our documentation](https://auth0.com/docs/configure/applications/application-grant-types) for more info and how to enable it.
*/
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

These get rendered duplicated.

@adamjmcgrath
Copy link
Copy Markdown
Contributor

I noticed that the code in the drop downs on the README isn't being highlighted, looks like an issue with jazzy

image

@Widcket
Copy link
Copy Markdown
Contributor Author

Widcket commented Dec 14, 2021

I noticed that the code in the drop downs on the README isn't being highlighted, looks like an issue with jazzy

image

Yup, that's a known issue: realm/jazzy#1116
We'll be replacing Jazzy with Apple's DocC next year anyway: swiftlang/swift-docc#56

@Widcket Widcket merged commit 276c219 into beta Dec 14, 2021
@Widcket Widcket deleted the v2/jazzy branch December 14, 2021 17:10
@cocojoe
Copy link
Copy Markdown
Member

cocojoe commented Dec 15, 2021

Looks cool

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:large Large review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants