Skip to content

style(website): apply ruff format to Python snippet in agents page - #3443

Merged
vpetersson merged 1 commit into
masterfrom
fix/ruff-format-agents-doc
Nov 3, 2026
Merged

vpetersson merged 1 commit into
masterfrom
fix/ruff-format-agents-doc

Conversation

@vpetersson-bot

Copy link
Copy Markdown
Contributor

Summary

uv run ruff format --check . fails on master with the pinned ruff (0.17.1), not because of any dependency bump. Ruff formats Python code blocks inside Markdown, and the Python snippet in website/content/agents.md (added in #3426) uses double quotes, while the repo's ruff config enforces single quotes.

The Python lint workflow is path-filtered to *.py, pyproject.toml and uv.lock, so it did not run on the docs change. The failure first showed up on later PRs that touch those paths, including the Dependabot PRs #3434, #3431 and #3430.

This PR runs ruff format on that file. The snippet is otherwise unchanged: quote style plus one blank line before the function definition.

Verification (local, ruff 0.17.1 from uv.lock)

🤖 Generated with Claude Code

The agent-facing docs page added a Python code block that ruff (which
formats Python in Markdown) wants in single-quote style. The lint
workflow is path-filtered to Python/dependency files, so it did not run
on that change and the failure only surfaced on later dependency PRs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings October 31, 2026 09:03
@vpetersson-bot
vpetersson-bot requested a review from a team as a code owner October 31, 2026 09:03

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error: Your billing is not configured or you have Copilot licenses from multiple standalone organizations or enterprises. To use premium requests, select a billing entity via the GitHub site, under Settings > Copilot > Features.

@sonarqubecloud

Copy link
Copy Markdown

@vpetersson
vpetersson merged commit 39d0d61 into master Nov 3, 2026
4 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.

3 participants