Skip to content

Tidy docstring placement in vector projection Poisson test - #257

Merged
lmoresi merged 1 commit into
underworldcode:developmentfrom
ss2098:ss2098/docstring-tidy-vector-projection
Jun 22, 2026
Merged

Tidy docstring placement in vector projection Poisson test#257
lmoresi merged 1 commit into
underworldcode:developmentfrom
ss2098:ss2098/docstring-tidy-vector-projection

Conversation

@ss2098

Copy link
Copy Markdown
Contributor

This follow-up tidies the docstring placement in test_vector_projection_after_poisson_with_units() so that the docstring remains the first statement in the function.

No solver or units-system logic is changed.

Tested with:

pixi run python -m pytest tests/test_1450_poisson_vector_projection.py -q

Result:

3 passed in 8.72s

@ss2098
ss2098 requested a review from lmoresi as a code ownerJune 19, 2026 07:55

@lmoresilmoresi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ss2098 — this is exactly the docstring-placement tidy I flagged on #227. Docstring is now the function's first statement again. Green CI. Merging.

@lmoresi
lmoresi merged commit 13e2eca into underworldcode:developmentJun 22, 2026
1 check passed
Sign up for freeto 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

@ss2098@lmoresi