Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: reformat multi-line statements #60

Merged

Conversation

mwtoews
Copy link
Contributor

@mwtoews mwtoews commented Nov 24, 2024

This PR refactors some multi-line statements. This was mostly automated by temporarily toggling this control in pyproject.toml:

[tool.ruff.format]
skip-magic-trailing-comma = true

then selecting some commits that condense multi-line statements to single, while rejecting other statements that are better represented over multiple lines.

@wpbonelli wpbonelli merged commit cab7c67 into MODFLOW-USGS:develop Nov 25, 2024
38 checks passed
@mwtoews mwtoews deleted the reformat-multi-line-statements branch November 25, 2024 16:55
@mwtoews
Copy link
Contributor Author

mwtoews commented Nov 25, 2024

Triple-check of changes complete. A future PR should amend the following to .git-blame-ignore-revs:

# reformat multi-line statements (#60)
cab7c679ffe255a8e73ab6f01b8faf5bdd4a4579

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