Skip to content

style: fix pre-existing lint debt in stratigraphic_column and model_exporter - #323

Merged
lachlangrose merged 1 commit into
masterfrom
chore/lint-cleanup-stratigraphic-column-model-exporter
Aug 24, 2026
Merged

style: fix pre-existing lint debt in stratigraphic_column and model_exporter#323
lachlangrose merged 1 commit into
masterfrom
chore/lint-cleanup-stratigraphic-column-model-exporter

Conversation

@lachlangrose

Copy link
Copy Markdown
Member

Summary

  • Applies black formatting to stratigraphic_column.py, which had drifted out of format over time.
  • Adds missing type hints to _model_exporter.py so it passes ruff's ANN ruleset (this file is not in the D/ANN grandfather list in pyproject.toml).

Split out from #322 to keep that PR's diff focused on its actual bug fix.

Test plan

  • black --check passes on both files
  • ruff check passes on both files
  • pytest tests/unit/modelling and tests/unit/io/test_exporters.py pass (225 passed, 2 skipped)

stratigraphic_column.py had accumulated pre-existing black formatting
drift. _model_exporter.py was missing type annotations required by
the ruff ANN ruleset (it is not in the D/ANN grandfather list).
@lachlangrose
lachlangrose merged commit 5261279 into master Aug 24, 2026
6 checks passed
@lachlangrose
lachlangrose deleted the chore/lint-cleanup-stratigraphic-column-model-exporter branch August 24, 2026 03:46
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.

1 participant