Skip to content

mkdocs to zensical github pages#126

Merged
0xba1a merged 1 commit intomainfrom
siva/zensical-docs-integration
Apr 3, 2026
Merged

mkdocs to zensical github pages#126
0xba1a merged 1 commit intomainfrom
siva/zensical-docs-integration

Conversation

@shivashanmugam
Copy link
Copy Markdown
Collaborator

Local verification of zensical-docs-integration
.
image

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.51%. Comparing base (f296252) to head (418aa27).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #126   +/-   ##
=======================================
  Coverage   96.51%   96.51%           
=======================================
  Files          28       28           
  Lines        1264     1264           
=======================================
  Hits         1220     1220           
  Misses         44       44           
Flag Coverage Δ
integration 58.86% <ø> (-0.32%) ⬇️
ollama_local 54.74% <ø> (ø)
slow-browser 44.85% <ø> (ø)
slow-other 66.53% <ø> (+1.42%) ⬆️
unit 85.60% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shivashanmugam shivashanmugam changed the title mkdocs to zensical mkdocs to zensical github pages Apr 2, 2026
Copy link
Copy Markdown
Collaborator Author

@shivashanmugam shivashanmugam left a comment

Choose a reason for hiding this comment

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

@microsoft-github-policy-service agree company="Microsoft"

@shivashanmugam
Copy link
Copy Markdown
Collaborator Author

@microsoft-github-policy-service agree company="Microsoft"

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the documentation site deployment pipeline to use the Zensical tool/theme instead of MkDocs Material for GitHub Pages publishing.

Changes:

  • Switch MkDocs theme from material to zensical in mkdocs.yml.
  • Update the GitHub Pages workflow to install zensical and run zensical build --clean instead of mkdocs build --strict.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
mkdocs.yml Changes the configured theme name to zensical for the docs site.
.github/workflows/pages.yml Changes the Pages build steps to install and build with zensical.
Comments suppressed due to low confidence (1)

mkdocs.yml:12

  • After switching the theme name to zensical, the rest of the theme: configuration still contains MkDocs Material–specific settings (e.g., features: navigation.instant, and toggle.icon: material/brightness-*). If Zensical doesn’t support these keys/icons, the docs build will fail at config validation or render incorrectly. Update the theme configuration to the Zensical-compatible options, or confirm Zensical is a Material-compatible theme and keep only supported keys.
  name: zensical
  palette:
    - media: "(prefers-color-scheme: light)"
      scheme: default
      primary: indigo

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +29 to +30
- name: Install Zensical
run: pip install zensical
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

The workflow now installs only zensical, but mkdocs.yml enables pymdownx.* markdown extensions. Unless zensical pulls those dependencies in transitively, the site build will fail due to missing Python modules. Consider installing the docs toolchain explicitly (e.g., the required markdown extension packages and any generator runtime) or using a dedicated requirements file for docs.

Suggested change
- name: Install Zensical
run: pip install zensical
- name: Install docs toolchain
run: pip install zensical pymdown-extensions

Copilot uses AI. Check for mistakes.
- name: Install MkDocs and dependencies
run: pip install mkdocs-material
- name: Install Zensical
run: pip install zensical
Copy link

Copilot AI Apr 3, 2026

Choose a reason for hiding this comment

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

Installing zensical without a version pin makes the Pages build non-reproducible and increases supply-chain risk (a newly released version could break the build). Pin the package version (or install from a locked requirements/constraints file) so GitHub Pages deployments are deterministic.

Suggested change
run: pip install zensical
run: pip install zensical==1.0.0

Copilot uses AI. Check for mistakes.
@0xba1a 0xba1a merged commit 39a77cd into main Apr 3, 2026
11 checks passed
@shivashanmugam shivashanmugam deleted the siva/zensical-docs-integration branch April 3, 2026 03:56
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.

4 participants