Fix CI: apply black formatting, add flake8 config, clean code quality issues#7
Merged
TUstudents merged 4 commits intomainfrom Feb 21, 2026
Merged
Conversation
… issues - Run black across all source and test files (27 files reformatted) - Add .flake8 config matching black's 88-char line length; suppress E501/E203/W503 and add per-file ignores for T201 (print in __main__ blocks) and E402 (matplotlib.use before imports in test_visualization) - Remove unused imports (Union, Tuple, j1, warnings, QUnit, IntensityUnit, pytest in conftest/test_basic) - Fix F541 f-string without placeholder in core.py __str__ - Remove unused local variable current_bounds_idx in fitting.py - Replace C408 dict() calls with dict literals in visualization.py - Fix E262 inline comment style in test_modeling.py - Add stacklevel=2 to warnings.warn in test_analysis.py (B028) - Update README.md: replace placeholder installation section with working from-source instructions and a Quick Start code example https://claude.ai/code/session_01PXC8wLEeZ6ZkLiXZCMVdUt
- Remove `wheel` from build-system requirements (setuptools>=61 handles it) - Add [tool.uv] dev-dependencies section to pyproject.toml - Update CI workflow to use astral-sh/setup-uv@v5 and `uv run` prefix - Update CLAUDE.md to document uv commands (uv sync --all-extras, uv run pytest, etc.) - Regenerate uv.lock after pyproject.toml changes https://claude.ai/code/session_01PXC8wLEeZ6ZkLiXZCMVdUt
- Replace setuptools build backend with uv_build>=0.8.0,<0.9.0 - Remove [tool.setuptools.packages.find] and [tool.setuptools.package-data] (uv_build auto-discovers packages and includes all files in the module root) - Add [tool.uv.build-backend] with module-root = "" for non-src layout - Wheel build confirmed: all sub-packages and example data present https://claude.ai/code/session_01PXC8wLEeZ6ZkLiXZCMVdUt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
E501/E203/W503 and add per-file ignores for T201 (print in main
blocks) and E402 (matplotlib.use before imports in test_visualization)
pytest in conftest/test_basic)
from-source instructions and a Quick Start code example
https://claude.ai/code/session_01PXC8wLEeZ6ZkLiXZCMVdUt