search

MDAnalysis / mdacli

Command line interface for MDAnalysis
https://mdacli.readthedocs.io/
GNU General Public License v3.0
19 stars 7 forks source link

Do not convert `None` when parsing analysis parameters #91

Closed PicoCentauri closed 2 years ago

pep8speaks commented 2 years ago

Hello @PicoCentauri! 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 2022-02-25 20:12:00 UTC
codecov[bot] commented 2 years ago

Codecov Report

Merging #91 (9414b17) into main (3ca8ec5) will increase coverage by 0.12%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
+ Coverage   82.87%   82.99%   +0.12%     
==========================================
  Files           8        8              
  Lines         543      547       +4     
  Branches      101      103       +2     
==========================================
+ Hits          450      454       +4     
  Misses         76       76              
  Partials       17       17
Impacted Files Coverage Δ
src/mdacli/libcli.py 97.52% <100.00%> (+0.04%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3ca8ec5...9414b17. Read the comment docs.