Skip to content

💚 Move ruff dependency to shared requirements-docs-tests.txt to fix "Build docs" workflow in CI#1458

Merged
tiangolo merged 1 commit into
masterfrom
move-ruff-dependency-to-requirements-docs-tests
Dec 23, 2025
Merged

💚 Move ruff dependency to shared requirements-docs-tests.txt to fix "Build docs" workflow in CI#1458
tiangolo merged 1 commit into
masterfrom
move-ruff-dependency-to-requirements-docs-tests

Conversation

@YuriiMotov

@YuriiMotov YuriiMotov commented Dec 23, 2025

Copy link
Copy Markdown
Member

Currently scripts/docs.py requires ruff, but requirements-docs.txt doesn't include it, so Build docs workflow fails

python ./scripts/docs.py verify-readme
...
Traceback (most recent call last):
  File "/home/runner/work/typer/typer/./scripts/docs.py", line 9, in <module>
    from ruff.__main__ import find_ruff_bin
ModuleNotFoundError: No module named 'ruff'

@YuriiMotov YuriiMotov added the internal Internal changes label Dec 23, 2025
@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview

Last commit 5217ce8 at: https://ad15d109.typertiangolo.pages.dev

@tiangolo tiangolo left a comment

Copy link
Copy Markdown
Member

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 c1806c4 into master Dec 23, 2025
30 checks passed
@tiangolo
tiangolo deleted the move-ruff-dependency-to-requirements-docs-tests branch December 23, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Internal changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants