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

Fix deploy site workflow #2113

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zachlewis
Copy link
Collaborator

New and improved github workflow for building and deploying our Hugo-based site.

Instead of maintaining a "gh-pages" branch, this workflow uses the preferred "deploy-site" GH action. This requires updating the repo settings (Code and Automation > Pages > Build and Deploy) to use the workflow-based deployment settings, as opposed to the classic branch-based deployment settings.

(NB: I've already updated the repo settings in anticipation of merging this PR).

Also, I haven't thoroughly tested this. The workflow is based on github's template for building and deploying Hugo sites, but if something goes wrong... well... I'll try again with another PR!

@zachlewis zachlewis force-pushed the fix_deploy_site_workflow branch from a541b58 to 061f231 Compare January 20, 2025 19:07
@doug-walker
Copy link
Collaborator

@zachlewis , currently the original OCIO v1 website is being maintained as part of the site at this link. We reference it on the landing page of the documentation here. Since that is in the gh_pages branch, and is not generated from Hugo, I'm guessing that would be dropped. Am I correct?

At this point, I'm not sure we still need to preserve the old website. Any opinions on that?

@zachlewis
Copy link
Collaborator Author

Ah, yes, I forgot to mention that. You are correct, we would no longer be preserving the old site with this workflow. If that's a concern, we can work something else out that still uses the "classic" gh-pages branch method.

@carolalynn
Copy link
Collaborator

carolalynn commented Jan 21, 2025 via email

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.

3 participants