Apply ruff formatting #3

Merged

forgejo_admin merged 2 commits from ruff-formatting into main

2026-03-01 02:32:07 +00:00

forgejo_admin commented

2026-02-27 20:51:09 +00:00

Owner

Summary

Mechanical ruff format . and ruff check --fix . pass to satisfy Woodpecker CI ruff format --check step
Part of plan-2026-02-25-mcp-gateway-migration Phase 2 (library pipelines)
2 remaining ruff check errors that need manual attention (see below)

Changes

9 files reformatted, 5 lint auto-fixes applied across src/ and tests/
Remaining errors (cannot auto-fix):
- F841 in tests/test_convenience.py:341 -- unused variable mixin (unsafe fix available via --unsafe-fixes)
- E741 in tests/test_labels.py:16 -- ambiguous variable name l

Test Plan

Woodpecker CI ruff format --check passes on this branch
No behavioral changes -- formatting only
2 remaining lint errors may need manual fix or noqa suppression

Review Checklist

No secrets committed
No unnecessary file changes
Commit messages are descriptive

plan-2026-02-25-mcp-gateway-migration -- Phase 2: Library Pipelines
issue-pal-e-services-library-pipelines

## Summary - Mechanical `ruff format .` and `ruff check --fix .` pass to satisfy Woodpecker CI `ruff format --check` step - Part of `plan-2026-02-25-mcp-gateway-migration` Phase 2 (library pipelines) - 2 remaining ruff check errors that need manual attention (see below) ## Changes - 9 files reformatted, 5 lint auto-fixes applied across `src/` and `tests/` - **Remaining errors (cannot auto-fix):** - `F841` in `tests/test_convenience.py:341` -- unused variable `mixin` (unsafe fix available via `--unsafe-fixes`) - `E741` in `tests/test_labels.py:16` -- ambiguous variable name `l` ## Test Plan - [ ] Woodpecker CI `ruff format --check` passes on this branch - [ ] No behavioral changes -- formatting only - [ ] 2 remaining lint errors may need manual fix or `noqa` suppression ## Review Checklist - [ ] No secrets committed - [ ] No unnecessary file changes - [ ] Commit messages are descriptive ## Related Notes - `plan-2026-02-25-mcp-gateway-migration` -- Phase 2: Library Pipelines - `issue-pal-e-services-library-pipelines`

forgejo_admin added 1 commit

2026-02-27 20:51:09 +00:00

Apply ruff formatting and auto-fixes

ci/woodpecker/pr/woodpecker Pipeline failed

Details

86b276be1e

Mechanical formatting pass to satisfy Woodpecker CI ruff format --check.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

forgejo_admin commented

2026-02-27 20:52:00 +00:00

Author

Owner

Review: Changes are mechanical ruff formatting plus 5 safe auto-fixes (unused imports removed, blank lines after imports). 2 remaining lint errors (F841 unused var, E741 ambiguous name) are pre-existing and non-blocking for the format check. Clean to merge.

forgejo_admin added 1 commit

2026-03-01 02:23:29 +00:00

Fix pre-existing ruff check errors

ci/woodpecker/pr/woodpecker Pipeline was successful

Details

a960455299

- F841: Remove unused `mixin` variable in test_convenience.py
- E741: Rename ambiguous variable `l` to `label` in test_labels.py

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>