-
Notifications
You must be signed in to change notification settings - Fork 224
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
Jazzy docs #190
Jazzy docs #190
Conversation
@croossin Need to do a little review, but it's at 100% documentation. A lot of it is redundant property name, or self explanatory property comments. Anything else you'd like done? |
Ready for any final comments and whether or not I should add in build step or leave it to you. |
Would be awesome to have as a build step @xtopolis - but how much of a lift is that? |
@xtopolis Is it hard to get this in the build pipeline? |
I'll be able to look into it by end of week/this weekend, sorry for the delay. |
@croossin Merge away if ready |
Before you make a Pull Request, read the important guidelines:
Issue Link 🔗
Goals of this PR 🎉
.jazzy.yaml
How Has This Been Tested 🔍
Please let us know if you have tested your PR and if we need to reproduce the issues. Also, please let us know if we need any relevant information for running the tests.
Test Configuration 👾
Things to check on 🎯