← Back to issue list

style: update to black 2024

View original Github issue

Metadata

Project
craft-parts
Number
#638
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged Black 2024 formatting updates to resolve CI failures. Approved by two reviewers with passing checks.

Suggested action:

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- Update to black 2024 style (automatic fixes simply by running `black`), which is causing [CI to fail](https://github.com/canonical/craft-parts/actions/runs/7702082875/job/20989575488?pr=627#step:5:12)

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged Black 2024 formatting updates to resolve CI failures. Approved by two reviewers with passing checks.
qwen3.6-35b-a3b-mtp-q6 Merged an update to format code using Black 2024, resolving CI failures caused by outdated styling. Automatic fixes applied via the Black formatter aligned the codebase with the new style guide.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#185 style(black): update black to v24 craft-application merged Merged update to Black v24, modifying pyproject.toml and auto-formatting four files. Approved by two reviewers, passed CI, and maintained 99.28% test coverage.
81%
#1491 style(autoformat): update black to 2024 charmcraft merged Merged following two approvals. Updated the Black formatter to the 2024 version, auto-formatting Python code across five files. Reported test failures were unrelated and traced to an external dependency issue.
80%
#3722 static: black update to 22 syntax snapcraft merged Merged update to Black formatter v22 syntax across 55 files. Approved by reviewer, passed CI, and integrated into main despite a minor 0.08% coverage decrease.
80%
#2599 static: update to newer black snapcraft merged Merged after reviewer approval and passing CI. Updated the codebase to a newer Black formatter version across 15 files (+35/-41 lines). Codecov reported 85.71% diff coverage, flagging a missing base commit upload.
79%
#2603 black: minor format changes from updated black snapcraft merged Minor formatting updates from the updated black code formatter were reviewed, approved, passed CI checks, and successfully merged into the main branch.
77%
#2180 code: use black as the standard style snapcraft merged Merged after approval and CI checks. Adopted Black as the standard code formatter, reformatting 567 files with 39,831 additions and 33,762 deletions. Diff coverage reached 90.92%.
75%
#315 style: use black for consistent and automated formatting charmcraft merged Merged after two approvals. Adopted Black for automated formatting across 36 files, added a formatting test, and configured flake8 to ignore conflicting rules. Increased the line length limit to accommodate Black's output.
75%
#554 Update black to 24 rockcraft closed Resolved through pull request #540. The Black formatter was updated to version 24, and the issue was closed after the PR was merged to maintain current code standards.
74%
#3865 lint: lint legacy code with black snapcraft merged Merged updates to the test-black lint command to include legacy code, aligning it with autoformat-black. Approved by a maintainer, passed CI checks, and merged without affecting code coverage.
73%
#3853 lint: lint legacy code with black snapcraft merged Merged following reviewer approval and passing CI. Applied Black formatting to legacy code across 12 files, resulting in 84 additions and 37 deletions. Code coverage remained stable at 88.16%.
72%