Skip to content

Add workflow for deploying GitHub pages - #8

Merged
awvwgk merged 2 commits into
microsoft:mainfrom
awvwgk:feat/docs
Oct 14, 2025
Merged

Add workflow for deploying GitHub pages#8
awvwgk merged 2 commits into
microsoft:mainfrom
awvwgk:feat/docs

Conversation

@awvwgk

@awvwgk awvwgk commented Oct 10, 2025

Copy link
Copy Markdown
Member

With this we can deploy GitHub pages directly from the actions pipeline, no need to have a separate branch with the source of the pages.

@awvwgk
awvwgk requested review from Copilot and tvogels October 10, 2025 17:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a GitHub Actions workflow to automatically deploy documentation to GitHub Pages from the main branch. The workflow builds Sphinx documentation and deploys it directly through GitHub Actions, eliminating the need for a separate pages source branch.

Key changes:

  • New automated docs deployment workflow with build and deploy jobs
  • Sphinx documentation building with HTML output
  • GitHub Pages deployment using official Actions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@awvwgk

awvwgk commented Oct 14, 2025

Copy link
Copy Markdown
Member Author

Deployed at https://awvwgk.github.io/skala/ via https://github.com/awvwgk/skala/actions/runs/18489571682

@awvwgk
awvwgk merged commit 60277ae into microsoft:main Oct 14, 2025
8 of 9 checks passed
@awvwgk
awvwgk deleted the feat/docs branch October 14, 2025 08:17
awvwgk added a commit that referenced this pull request Nov 7, 2025
- When creating the density fitting object the xc method must be a
  known libxc functional to determine whether it is a hybrid
- Calling `predefined_auxbasis(..., xc="tpss")` before applying is
  not possible density fitting as this routine returns always None
  for non-hybrids
- Fix this by temporarily setting `xc = "tpss"` before applying
  density fitting and revert to `xc = "custom"` to restore guard
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