Skip to content

Format the operator docs test (fixes main lint after #324) - #325

Merged
fredesere merged 1 commit into
mainfrom
fix/format-docs-test
Nov 3, 2026
Merged

fredesere merged 1 commit into
mainfrom
fix/format-docs-test

Conversation

@fredesere

@fredesere fredesere commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Main's Lint the thin subnet tree failed after #324: Would reformat: tests/thin/test_operator_docs_safety.py. This is ruff format only; there is no logic change. ruff format --check tests/thin is clean and the docs tests pass (15).


Note

Low Risk
Formatting-only edits in a test file with no runtime or behavioral impact.

Overview
Fixes main lint after #324 by running ruff format on tests/thin/test_operator_docs_safety.py only.

The diff wraps one long assert string check across multiple lines and reformats the OPERATOR_LINK_DOCUMENTS list with a trailing comma. No test logic or assertions change—behavior stays the same; ruff format --check tests/thin is expected to pass again.

Reviewed by Cursor Bugbot for commit c017efc. Bugbot is set up for automated code reviews on this repo. Configure here.

@fredesere
fredesere merged commit de71dfe into main Nov 3, 2026
8 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-03T03:38:53.027251Z c017efc PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@fredesere
fredesere deleted the fix/format-docs-test branch November 3, 2026 04:07
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.

1 participant