Skip to content

Conversation

@marcarl
Copy link
Collaborator

@marcarl marcarl commented Jan 4, 2026

Summary

This PR addresses PEP 8 compliance issues throughout the codebase to ensure the project follows the coding standards documented in CONTRIBUTING.md.

Changes

Whitespace and formatting (900+ fixes)

  • ✅ Remove trailing whitespace from all lines (W291, W293)
  • ✅ Fix newline issues at end of files (W292, W391)
  • ✅ Fix blank lines between functions (E302, E303, E305)
  • ✅ Fix block comment formatting (E265)

Line length (60 fixes)

  • ✅ Break long lines where possible using autopep8 (E501)
  • ✅ Reduced from 174 to 114 E501 violations
  • ⚠️ Remaining violations are long strings/comments per CONTRIBUTING.md flexibility

Indentation (20 fixes)

  • ✅ Fix continuation line indentation (E128, E129)

Statistics

  • Total fixes: 900+ PEP 8 violations resolved
  • Files modified: 49 Python files
  • Lines changed: +1939, -1594

Remaining issues

114 E501 (line too long) violations remain, primarily for:

  • Long string literals
  • Complex regex patterns
  • Descriptive comments

These are acceptable per CONTRIBUTING.md line 59: "Maximal radlängd: 100 tecken (flexibelt för långa strängar)"

Verification

All tests pass with the changes. PEP 8 compliance can be verified with:

pycodestyle --max-line-length=100 --statistics temporal/ formatters/ exporters/ util/ sfs_processor.py

🤖 Generated with Claude Code

This commit addresses PEP 8 compliance issues throughout the codebase to ensure
the project follows the coding standards documented in CONTRIBUTING.md.

## Changes made:

### Whitespace and formatting (900+ fixes)
- Remove trailing whitespace from all lines (W291, W293)
- Fix newline issues at end of files (W292, W391)
- Fix blank lines between functions (E302, E303, E305)
- Fix block comment formatting (E265)

### Line length (60 fixes)
- Break long lines where possible using autopep8 (E501)
- Reduced from 174 to 114 E501 violations
- Remaining violations are long strings/comments per CONTRIBUTING.md flexibility

### Indentation (20 fixes)
- Fix continuation line indentation (E128, E129)

## Remaining issues:
- 114 E501 (line too long) violations remain, primarily for:
  - Long string literals
  - Complex regex patterns
  - Descriptive comments

These are acceptable per CONTRIBUTING.md: "Maximal radlängd: 100 tecken
(flexibelt för långa strängar)"

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

2 participants