mosdef-hub / mbuild

A hierarchical, component based molecule builder
https://mbuild.mosdef.org
Other
173 stars 81 forks source link

Fix deprecation warnig in `packing.py` and replace `.format()` with f-strings where possible. #1192

Closed chrisjonesBSU closed 3 months ago

chrisjonesBSU commented 4 months ago

PR Summary:

This is a PR that does 2 things:

1) Fix a deprecation warning that is being raised about find_executable in distutils

>>> from distutils.spawn import find_executable
>>> find_executable("PACKMOL")
<stdin>:1: DeprecationWarning: Use shutil.which instead of find_executable

2) It also replaces .format() with f-strings in many places in an effort to clean things up a bit (I was bored at an airport)

PR Checklist


codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.34%. Comparing base (8649337) to head (e120962). Report is 39 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1192 +/- ## ========================================== - Coverage 87.35% 87.34% -0.01% ========================================== Files 62 62 Lines 6586 6584 -2 ========================================== - Hits 5753 5751 -2 Misses 833 833 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

chrisjonesBSU commented 3 months ago

Merging this after discussing with Cal on a mosdef dev call.