-
Notifications
You must be signed in to change notification settings - Fork 9
docs: enable SPI documentation rendering via DocC #125
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.spi.yml drives the documentation engine:
| ## Getting Started | ||
|
|
||
| TODO: We should refer auto generated documentation on Swift Package Index after we register this package to Swift Package Index. | ||
| For API documentation and usage examples, see the [documentation on Swift Package Index](https://swiftpackageindex.com/apache/arrow-swift/documentation). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That link will start to work after the merge (when SPI picks up the changes)
|
Could you add our license header? |
|
@kou I added the header from the README to the documentation index file. Finger crossed Apple's weird build tools work with that! |
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
|
Thanks @kou I didn't think of the yaml file! |
kou
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
What's Changed
Swift Package Index automatically hosts rendered and browse-able documentation when DocC is used.
This PR adds an overview of the API surface of this library (a Markdown index file) that guides the documentation generator. It can be browsed in Xcode, too, if you want a preview:
Enhancements would include more tutorialized content on how to use this, but the project website exists for this for now.
Closes #128.