-
Notifications
You must be signed in to change notification settings - Fork 38
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
Publish Swift SymbolKit's documentation to GitHub pages #19
Conversation
@swift-ci please test |
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.
This looks great! Thanks so much for putting this together @Kyle-Ye.
I went ahead and used the script you included to deploy to my fork and it looks like everything is working as expected: https://ethan-kusters.github.io/swift-docc-symbolkit/documentation/symbolkit/.
0028e33
to
acf0876
Compare
@swift-ci please test |
By the way, swift-docc-plugin has a dependency on main branch of |
acf0876
to
99c820e
Compare
@swift-ci please test |
That is a great question... Let me check. |
@swift-ci please test macOS |
The macOS Platform CI just exits directly🤔. Could you help confirm with it? @ethan-kusters |
This comment was marked as outdated.
This comment was marked as outdated.
@swift-ci please test |
It looks like it does:
Hm. I think we'll need to figure out a workaround here... It might be we just need to add a custom script for building documentation instead of using the plugin. I'm intentionally trying to keep the number of dependencies the plugin has a to a minimum for this exact reason but SymbolKit is unavoidable at the moment. |
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.
Unfortunately I don't think we can land this as-is since it will introduce a cyclic dependency with the Swift-DocC Plugin.
Got it, seems like the dependency was introduced 3 days ago by swiftlang/swift-docc-plugin#7. So there is 2 ways to solve this:
Personally I prefer way 1, but way 2 is acceptable for me. Can I get your suggestion on this? @ethan-kusters cc @bitjammer |
@bitjammer can speak to this more but my understanding is in the longer-term we'd like to move the new So I think we should plan on using a custom script instead of the Swift-DocC Plugin for now but plan on removing it and adopting the Plugin as soon as it becomes possible. I've written a couple of similar scripts before, so I'm happy to open a PR against your branch later today to add the basis for something and we can iterate on it there. |
So unfortunately, moving the tool to a new repo will not solve the dependency problem as SwiftPM plugins share their dependency graph with their clients. This is not an intentional limitation and ought to be fixed but I don't see it happening very soon. We can't remove the dependency on SymbolKit because we use the Symbol Graph model to communicate snippet information to DocC. I'm seeing what my options are for deploying the tool elsewhere but if we can have this one exception in the meantime, I would appreciate it! |
@Kyle-Ye I've opened a PR against your branch here with a script as a temporary workaround until we resolve these issues with the plugin: Feel free to merge it directly into this PR if it looks good to you. |
Temporarily removes the Swift-DocC Plugin since we have a depenency cycle issue.
@swift-ci please test |
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.
Looks great! Thanks for all your work on this @Kyle-Ye.
The doc page is currently unavailable https://apple.github.io/swift-docc-symbolkit/documentation/symbolkit/ Seems that we need to create/enable the some environment like the following. cc @franklinsch |
It's live now: https://apple.github.io/swift-docc-symbolkit/documentation/symbolkit. Thanks for the amazing work here once again @Kyle-Ye 🚀 |
Summary
This sets the foundation for deploying Swift SymbolKit's documentation to GitHub pages.
After merging this PR, we should be able to prepare a
gh-pages
branch on the mainswift-docc-symbolkit
repo with no content on it, run thebin/update-gh-pages-documentation-site
script and have docs available.Dependencies
None.
Testing
With Swift 5.6, run:
and confirm the Swift-DocC preview works as expected.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.