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

Build and Host Website on Netlify #528

Merged
merged 1 commit into from
Jul 23, 2021
Merged

Conversation

akestner
Copy link
Contributor

Issue, if available:
N/A

Description of changes:
After wrestling with Github Pages for the last 24 hours, and failing to get it to play nicely with Karpenter's monorepo structure and Hugo, I propose we build and host the docs site on Netlify. Over time we can evolve this approach as needed, but for the near term, I believe this is good solution for the project.

This PR includes two changes, first is a netlify.toml config file that is essentially copied from the Hugo Netlify docs. The second change is a minor change to the Hugo configuration file to simplify the baseURL and remove the publishDir as its no longer needed.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

ellistarn
ellistarn previously approved these changes Jul 22, 2021
@geoffcline
Copy link
Contributor

Looks great to me!

I appreciate that with this strategy we won't need to worry about the Hugo generated HTML in the repo.

@akestner
Copy link
Contributor Author

Rebased off main, apologies for the force push

@akestner akestner requested a review from ellistarn July 23, 2021 18:44
@geoffcline geoffcline self-requested a review July 23, 2021 19:02
@akestner akestner merged commit 70b498d into aws:main Jul 23, 2021
gfcroft pushed a commit to gfcroft/karpenter-provider-aws that referenced this pull request Nov 25, 2023
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

3 participants