Skip to content

Gem warning, page layouts, old pages, general fixes #208

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

Merged
merged 9 commits into from
Jun 14, 2025

Conversation

Langhaarzombie
Copy link
Contributor

This PR resolves the warning that occurs when building the webpage using GitHub Actions.
The change includes a transition from using the branch based deployment method towards using the Jekyll plugin for GitHub Actions directly.
This closely followed the best practice suggested on the Jekyll website.

As part of this change, the settings for deployment need to be changed. This needs to be done by an admin and is described here.
Essentially, by going to Settings > Code and automation > Pages, the Build and Deployment source needs to be changed to GitHub Actions. This needs to be done after merging, so that the jekyll workflow can be selected.

Let me know if we should do a meeting for this transition!

Also fixes the 404 page and default layout...

Copy link
Contributor

@hodgestar hodgestar left a comment

Choose a reason for hiding this comment

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

Thanks! It's nice to have a bit more visibility / control over the deployment process.

@hodgestar
Copy link
Contributor

I've applied the admin settings update.

@Langhaarzombie Langhaarzombie changed the title Fix Gem warning on page build & default layout Gem warning, page layouts, old pages, general fixes Jun 10, 2025
@Langhaarzombie
Copy link
Contributor Author

I am turning this into a more general PR cause there are so many small things that are changed.

So to summarize the changes that are performed:

  • Fixes the Gem warning on page build (already reviewed)
  • Fixes the default layout of pages, 404 page looks good (already reviewed)
  • Adds message to dowloads page that it is archived
  • Readds the documentation page to the navbar for older versions
  • Fixes a href for the supporting organisations which is sometimes used on external pages

@hodgestar hodgestar merged commit f66a5a8 into qutip:master Jun 14, 2025
2 checks passed
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.

2 participants