Skip to content

⬆️ Increase lower version bound of pymdown-extensions to >=10.21.2#60

Merged
tiangolo merged 1 commit intotiangolo:mainfrom
YuriiMotov:fix-tests-fail-with-resolution-lowest
Apr 3, 2026
Merged

⬆️ Increase lower version bound of pymdown-extensions to >=10.21.2#60
tiangolo merged 1 commit intotiangolo:mainfrom
YuriiMotov:fix-tests-fail-with-resolution-lowest

Conversation

@YuriiMotov
Copy link
Copy Markdown
Contributor

Latest pygments release is incompatible with [pymdown-extensions](https://github.com/facelessuser/pymdown-extensions) < 10.21.2.

This causes tests in CI to fail with lowest-direct uv resolution (pymdown-extensions is direct dependency, so uv installs lowest version, but pygments is not a direct dependency, so uv installs latest version)..

In this PR I updated lower bound of pymdown-extensions. I don't know if it's not too strict..

We have at least 2 more options:

  • Pin pygments to < 2.20.0
  • Add pygments to direct dependencies

Copy link
Copy Markdown
Collaborator

@svlandeg svlandeg left a comment

Choose a reason for hiding this comment

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

Ugh, this is somewhat annoying because really we shouldn't have to do anything, the dependency resolution should just work. It feels to me more like a bug in the "lowest-direct" resolution 😕

That said, I don't think it matters all that much in the end, and bumping like you do here is probably the best resolution. Either way let's get this fixed so the CI goes green again.

Copy link
Copy Markdown
Owner

@tiangolo tiangolo left a comment

Choose a reason for hiding this comment

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

Thank you! 🙌

@tiangolo tiangolo merged commit 8ed1bb1 into tiangolo:main Apr 3, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file internal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants