Skip to content

Conversation

@Soliprem
Copy link
Collaborator

Sanity Checking

  • I have updated the changelog as per my changes
  • I have tested, and self-reviewed my code
  • My changes fit guidelines found in hacking nvf
  • Style and consistency
    • I ran Alejandra to format my code (nix fmt)
    • My code conforms to the editorconfig configuration of the project
    • My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • My code includes comments in particularly complex areas
    • I have added a section in the manual
    • (For breaking changes) I have included a migration guide
  • Package(s) built:
    • .#nix (default package)
    • .#maximal
    • .#docs-html (manual, must build)
    • .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin

Add a 👍 reaction to pull requests you find important.

same as before. New root seems to be share/doc/, and not share/doc/nvf
@NotAShelf NotAShelf changed the title workflows: fix the ci for deployment of the docs website ci: fix documentation deployment & preview Dec 12, 2025
@NotAShelf NotAShelf enabled auto-merge December 12, 2025 06:36
@NotAShelf NotAShelf disabled auto-merge December 12, 2025 06:36
@NotAShelf NotAShelf merged commit cc985cc into main Dec 12, 2025
14 of 15 checks passed
@NotAShelf NotAShelf deleted the fix-ci branch December 12, 2025 06:36
@github-actions
Copy link

✅ Preview has been deleted successfully!

github-actions bot pushed a commit that referenced this pull request Dec 12, 2025
@github-actions
Copy link

🚀 Live preview deployed from cc985cc

View it here:

Debug Information

Triggered by: NotAShelf

HEAD at: fix-ci

Reruns: 1677

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants