Add deprecation warning for ITPParser

lilyminium commented 1 month ago

Fixes #4698

Changes made in this Pull Request:

Adds deprecation warning for element guessing in ITPParser

Please feel free to make any changes desired and push into this branch!

PR Checklist

[x] Tests?
[x] Docs?
[x] CHANGELOG updated?
[x] Issue raised/referenced?

Developers certificate of origin

[x] I certify that this contribution is covered by the LGPLv2.1+ license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

📚 Documentation preview 📚: https://mdanalysis--4744.org.readthedocs.build/en/4744/

pep8speaks commented 1 month ago

Hello @lilyminium! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2024-10-24 18:11:49 UTC

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 93.62%. Comparing base (9b69745) to head (64804b6). Report is 2 commits behind head on develop.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## develop #4744 +/- ## =========================================== - Coverage 93.65% 93.62% -0.03% =========================================== Files 175 187 +12 Lines 21564 22631 +1067 Branches 3023 3023 =========================================== + Hits 20195 21189 +994 - Misses 925 998 +73 Partials 444 444 ```

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

lilyminium commented 1 month ago

Thanks Irfan, committed the changes you suggested!

MDAnalysis / mdanalysis