Skip to content

Conversation

@janosh
Copy link
Member

@janosh janosh commented Oct 2, 2023

The breaking part of this PR is f09769b which probably sees little use, hence no deprecation period. Can always revert if that turns out to be wrong.

696008f replace _parse_string with str.strip in PotcarSingle.parse_functions
328ae47 use keyword args to dict.update()
e9c7bee fix ruff
66304bc convert (point_ops|shared_ops).pkl to json.gz in tests/files/site_symmetries (better to avoid pickle in tests, JSON less brittle)
f09769b breaking: ETSF_Reader PascalCase to EtsfReader
cfdb96f fix TestNEBAnalysis.runTest renamed test_run so pytest discovers and actually runs it
1da4729 snake_case test method names
b09647f delete get_path helper in test_surface_analysis.py
61ec39f fix doc string indent
14d32fc no need for os.path.join in tests

@janosh janosh added tests Issues with or changes to the pymatgen test suite housekeeping Moving around or cleaning up old code/files breaking Breaking change labels Oct 2, 2023
@janosh janosh enabled auto-merge (squash) October 2, 2023 21:33
@janosh janosh merged commit 7fe4f7b into master Oct 2, 2023
@janosh janosh deleted the fix-undiscovered-tests branch October 2, 2023 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change housekeeping Moving around or cleaning up old code/files tests Issues with or changes to the pymatgen test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants