Skip to content

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

Merged
tiangolo merged 1 commit into
tiangolo: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 into
tiangolo: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

@svlandeg svlandeg left a comment

Copy link
Copy Markdown
Collaborator

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.

@tiangolo tiangolo left a comment

Copy link
Copy Markdown
Owner

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