← Back to issue list

test: remove black linting from test_infra

View original Github issue

Metadata

Project
charmcraft
Number
#1331
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged #1331 as d3afd159, removing 13 lines of Black linting from test_infra; tox now handles linting for CRAFT-2143.

Resolution: closed resolved

Reason: PR #1331 was approved and merged as commit d3afd159. It removed the test_infra-specific Black configuration and delegated linting to tox under CRAFT-2143.

Evaluated by: mr-cal-bot

No scores available.

Issue body

We're using tox for black linting now. CRAFT-2143

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #1331 as d3afd159, removing 13 lines of Black linting from test_infra; tox now handles linting for CRAFT-2143.
stealth/space-bunny-alpha — closed resolved Merged PR #1331 (d3afd159), removing 13 lines of Black linting configuration from test_infra so tox handles linting for CRAFT-2143.
qwen/qwen3.6-35b-a3b — — Merged after removing black linting configuration from test_infra. Linting now uses tox per CRAFT-2143. Approved by two reviewers, deleting 13 lines across one file.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Removes black linting from test_infra since tox now handles it. Resolves CRAFT-2143.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#4018 Remove non-tox CI tests and linting snapcraft merged Merged #4018 as commit 88fbd5b71, deleting 90 lines of redundant non-tox CI test and lint jobs from .github/workflows/tests.yaml after tox-based checks landed in #4005.
72%