-
Notifications
You must be signed in to change notification settings - Fork 160
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
Documentation with Dokka and KDoc #537
Conversation
@Reedyuk what do you think is the best way to publish the documentation? is there a GitHub Action we can use to publish the dokka output to github pages? @BasBuijsen do you have experience with publishing to GitHub pages? |
@BasBuijsen build failed due to the function you made internal that are needed by inline functions (can use @PublishedApi here) |
@nbransby i dont have experience with publishing to github pages but from what ive seen it is very easy. There is a ready to go action that will do all the magic if you set the environment variables correctly, so i think that would be the easiest and probably cheapest solution (there is one website free per gh organisation or user). I will fix the JS build this afternoon, after work :) |
…rebase-kotlin-sdk into feat/Documentation
…n-sdk into feat/Documentation
@Reedyuk thanks for adding the deployment steps! Ive added documentation for nearly all modules now. Only authentication, analytics and firestore are left to do. |
I think rebase didnt work correctly, as its not compiling on the github servers. |
…n-sdk into feat/Documentation
…rebase-kotlin-sdk into feat/Documentation
Yes i made a mistake while resolving the merge conflict. Will fix it :) |
get this building on the ci and i will merge in and make sure the docs deploy correctly. |
@BasBuijsen - can you rebase this pull request please |
@Reedyuk when its deployed can you send me the url so i can add it to the readme when i create a new pr with the final documentation changes next week? |
This adds the setup of Dokka. This generates HTML based on the KDoc comments provided for every function and class. For now i didn't add them yet to all the classes since that is a huge chunk of work. I will incrementally add documentation, but for now only showing the signatures should suffice.
The publishing of the generated HTML still needs to be setup. I need @nbransby for that so that is out of my control.
Since all publicly available classes and functions will be shown in the documentation, this change will also improve the documentation a lot. It will make sure that only the classes and functions that are actually exposed will be documented.