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

Change docs target #20

Merged
merged 2 commits into from
Apr 21, 2022
Merged

Conversation

finestructure
Copy link
Contributor

As discussed, this

  • removes the extra package manifest in Documentation
  • flattens the directory structure in Documentation
  • adds the SwiftBundler documentation target to the main manifest
  • updates update-docs.yml to the new directory structure

I don't think there's a way to test update-docs.yml prior to merging into main, is there?

Copy link
Owner

@stackotter stackotter left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, looks good 👍 Sadly I don't think there's a way to test the workflow before merging, so we'll just have to cross our fingers 🤞 It looks like it should work

@stackotter stackotter merged commit 19d38c6 into stackotter:main Apr 21, 2022
@finestructure
Copy link
Contributor Author

Looks like it worked, nice!

@stackotter
Copy link
Owner

stackotter commented Apr 21, 2022

I'll be very excited to see documentation generation added to SPI, so good luck! I've always been jealous of rust's cargo for that

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

2 participants