Skip to content

fix: pin mkdocs<2.0 to restore Material theme on docs site#114

Merged
eurunuela merged 2 commits intoME-ICA:masterfrom
eurunuela:fix/pin-mkdocs-version
Mar 3, 2026
Merged

fix: pin mkdocs<2.0 to restore Material theme on docs site#114
eurunuela merged 2 commits intoME-ICA:masterfrom
eurunuela:fix/pin-mkdocs-version

Conversation

@eurunuela
Copy link
Collaborator

Summary

Problem

pip install mkdocs-material (unpinned) now resolves to mkdocs 2.0, which is incompatible with mkdocs-material 9.x. The build technically "succeeds" but produces broken HTML — the deployed site has a Jekyll meta tag and none of the Material CSS/JS, making it unusable.

Fix

One-line change: add "mkdocs<2.0" to the pip install command so mkdocs-material continues to work with a compatible mkdocs 1.x version.

Test plan

🤖 Generated with Claude Code

mkdocs 2.0 is incompatible with mkdocs-material 9.x. When installed
unpinned, pip now resolves to mkdocs 2.0 which causes the build to
produce broken HTML with a Jekyll meta tag instead of the Material theme.

Pinning mkdocs<2.0 ensures the docs site renders correctly with the
Material theme.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 3, 2026 17:21
@netlify
Copy link

netlify bot commented Mar 3, 2026

Deploy Preview for rica-fmri ready!

Name Link
🔨 Latest commit d8b317a
🔍 Latest deploy log https://app.netlify.com/projects/rica-fmri/deploys/69a71a925817d20008329cf2
😎 Deploy Preview https://deploy-preview-114--rica-fmri.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
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

Pins MkDocs in the documentation GitHub Actions workflow to keep the MkDocs Material theme working and restore correct rendering on the published docs site.

Changes:

  • Update the docs workflow install step to add a mkdocs<2.0 constraint while installing mkdocs-material.

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


- name: Install dependencies
run: pip install mkdocs-material
run: pip install mkdocs-material "mkdocs<2.0"
Copy link

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

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

mkdocs-material is still unpinned here, so the workflow can break again when a new major/minor release of mkdocs-material changes its MkDocs compatibility. Consider pinning mkdocs-material to a known-compatible range (or using a checked-in requirements/constraints file) alongside mkdocs<2.0 to make the docs build reproducible.

Suggested change
run: pip install mkdocs-material "mkdocs<2.0"
run: pip install "mkdocs-material>=9,<10" "mkdocs<2.0"

Copilot uses AI. Check for mistakes.
Per review feedback: mkdocs-material was still unpinned, meaning a future
major release could break the build again. Pin to the 9.x series alongside
mkdocs<2.0.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Collaborator Author

@eurunuela eurunuela left a comment

Choose a reason for hiding this comment

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

Addressed — is now also pinned to .

@eurunuela eurunuela merged commit 2d7bdb3 into ME-ICA:master Mar 3, 2026
5 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