Skip to content

Rewrite CI workflows for Hugo - #967

Open
MuhammadAashirAslam wants to merge 3 commits into
precice:hugofrom
MuhammadAashirAslam:hugo-build-workflows
Open

Rewrite CI workflows for Hugo#967
MuhammadAashirAslam wants to merge 3 commits into
precice:hugofrom
MuhammadAashirAslam:hugo-build-workflows

Conversation

@MuhammadAashirAslam

Copy link
Copy Markdown
Collaborator

Migrated build.yml, deploy.yml, and testing.yml from Jekyll/Ruby to Hugo.

  • Builds with Hugo Extended, Go modules, Doxygen, and full Git history.

  • Validates Hugo RSS, sitemap, sitemap index, robots.txt, and Doxygen output.

  • Deploys production through GitHub Pages using the required pages: write and id-token: write permissions.
    GitHub Pages custom workflows

  • Replaced external testing-site publishing with a downloadable testing-site artifact. GitHub documents
    artifacts as the standard way to preserve and download workflow output after a run. Workflow artifacts

  • Removed the old Jekyll deployment token requirement; production deployment uses GitHub Pages’ scoped
    permissions instead.

This comment was marked as off-topic.

@MakisH MakisH left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am not sure if this PR is up-to-date: I see a few comments by Copilot in the Files changed view, which are not answered.

Other than that, deploying the hugo branch to https://precice.github.io/website-testing would be very helpful indeed. However, this contradicts the PR description.

Comment thread .github/workflows/deploy.yml
Comment thread .github/workflows/deploy.yml
@MuhammadAashirAslam

Copy link
Copy Markdown
Collaborator Author

Copilot comments were stale or irrevelant (so i marked them irrevelant).
On the website-testing side , the testing.yml was deploying the website to https://github.com/precice/website-testing which have been archived .

@MakisH

MakisH commented Aug 10, 2026

Copy link
Copy Markdown
Member

From our discussion:

  • Write a comment on the Copilot suggestions, linking to the related documentation,
  • We probably can drop the testing.yml workflow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

technical Technical issues on the website

Projects

Development

Successfully merging this pull request may close these issues.

3 participants