Skip to content
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 Markdown's documentation to GitHub pages #32

Merged
merged 8 commits into from
Mar 21, 2022

Conversation

ethan-kusters
Copy link
Contributor

@ethan-kusters ethan-kusters commented Mar 19, 2022

Summary

This sets the foundation for deploying Swift Markdown's documentation to GitHub pages.

  • Adds the Swift-DocC Plugin as a dependency of Swift-Markdown.

  • Adds a script to simplify deployment of documentation to GitHub pages.

  • Moves detailed documentation from the README into the DocC catalog.

After merging this PR, we should be able to prepare a gh-pages branch on the main swift-markdown repo with no content on it, run the bin/update-gh-pages-documentation-site script and have docs available.

Dependencies

None.

Testing

With Swift 5.6, run:

swift package --disable-sandbox preview-documentation --target Markdown

and confirm the Swift-DocC preview works as expected.

I've deployed the documentation to the GitHub pages site on my fork using the included script. Confirm the site behaves as expected here: https://ethan-kusters.github.io/swift-markdown/documentation/markdown/

Checklist

Make sure you check off the following items. If they cannot be completed, provide a reason.

  • Added tests
  • Ran the ./bin/test script and it succeeded
  • Updated documentation if necessary

@ethan-kusters ethan-kusters requested review from franklinsch, Kyle-Ye and QuietMisdreavus and removed request for franklinsch March 19, 2022 16:39
@ethan-kusters
Copy link
Contributor Author

@swift-ci please test

README.md Show resolved Hide resolved
@@ -26,3 +26,5 @@

## See Also
- <doc:BlockDirectives>

<!-- Copyright (c) 2021-2022 Apple Inc and the Swift Project authors. All Rights Reserved. -->
Copy link
Contributor

Choose a reason for hiding this comment

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

I feel like these missing license headers should get their own PR that can be reviewed and merged quickly - my own outstanding PRs here also have commits that add these 😅

@ethan-kusters
Copy link
Contributor Author

@swift-ci please test

Copy link
Contributor

@franklinsch franklinsch left a 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 doing this!

@ethan-kusters ethan-kusters merged commit e506935 into swiftlang:main Mar 21, 2022
@ethan-kusters ethan-kusters deleted the adopt-swift-docc-plugin branch March 21, 2022 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants