MDAnalysis / MDAnalysisData

Access to data for workshops and extended tests of MDAnalysis.
https://www.mdanalysis.org/MDAnalysisData
BSD 3-Clause "New" or "Revised" License
14 stars 5 forks source link

Try to modernise package a bit #61

Closed IAlibay closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

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

Comparison is base (98c6b34) 56.53% compared to head (eb47d43) 56.06%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #61 +/- ## ========================================== - Coverage 56.53% 56.06% -0.47% ========================================== Files 12 12 Lines 375 371 -4 Branches 49 51 +2 ========================================== - Hits 212 208 -4 Misses 160 160 Partials 3 3 ``` | [Files](https://app.codecov.io/gh/MDAnalysis/MDAnalysisData/pull/61?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MDAnalysis) | Coverage Δ | | |---|---|---| | [MDAnalysisData/\_\_init\_\_.py](https://app.codecov.io/gh/MDAnalysis/MDAnalysisData/pull/61?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MDAnalysis#diff-TURBbmFseXNpc0RhdGEvX19pbml0X18ucHk=) | `100.00% <100.00%> (ø)` | | | [MDAnalysisData/base.py](https://app.codecov.io/gh/MDAnalysis/MDAnalysisData/pull/61?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MDAnalysis#diff-TURBbmFseXNpc0RhdGEvYmFzZS5weQ==) | `98.30% <100.00%> (-0.03%)` | :arrow_down: | | [MDAnalysisData/datasets.py](https://app.codecov.io/gh/MDAnalysis/MDAnalysisData/pull/61?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=MDAnalysis#diff-TURBbmFseXNpc0RhdGEvZGF0YXNldHMucHk=) | `100.00% <ø> (ø)` | |

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

pep8speaks commented 1 year ago

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

Line 381:80: E501 line too long (84 > 79 characters) Line 471:80: E501 line too long (80 > 79 characters) Line 474:80: E501 line too long (84 > 79 characters)

Line 26:80: E501 line too long (80 > 79 characters) Line 27:80: E501 line too long (86 > 79 characters) Line 28:80: E501 line too long (81 > 79 characters) Line 33:80: E501 line too long (83 > 79 characters) Line 45:80: E501 line too long (83 > 79 characters) Line 222:80: E501 line too long (87 > 79 characters) Line 224:80: E501 line too long (86 > 79 characters) Line 252:80: E501 line too long (85 > 79 characters) Line 284:80: E501 line too long (87 > 79 characters) Line 286:80: E501 line too long (83 > 79 characters) Line 288:80: E501 line too long (86 > 79 characters) Line 888:80: E501 line too long (85 > 79 characters) Line 978:80: E501 line too long (80 > 79 characters) Line 981:80: E501 line too long (87 > 79 characters) Line 1419:80: E501 line too long (84 > 79 characters) Line 1620:80: E501 line too long (80 > 79 characters) Line 1623:80: E501 line too long (84 > 79 characters) Line 1893:80: E501 line too long (83 > 79 characters) Line 1988:80: E501 line too long (80 > 79 characters) Line 1989:80: E501 line too long (85 > 79 characters) Line 2030:80: E501 line too long (84 > 79 characters) Line 2032:80: E501 line too long (83 > 79 characters)

Comment last updated at 2023-10-29 06:54:50 UTC