cmelab / planckton

Reproducible exploration of organic photovoltaic morphology self-assembly using molecular dynamics.
GNU General Public License v3.0
5 stars 6 forks source link

clean up planckton and add linting checks to CI #42

Closed jennyfothergill closed 3 years ago

jennyfothergill commented 3 years ago

fixes #25

Added:

Need to add:

codecov[bot] commented 3 years ago

Codecov Report

Merging #42 (9da6fc4) into master (8dd4ee6) will increase coverage by 0.09%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #42      +/-   ##
==========================================
+ Coverage   98.32%   98.42%   +0.09%     
==========================================
  Files           5        6       +1     
  Lines         179      190      +11     
==========================================
+ Hits          176      187      +11     
  Misses          3        3              
Impacted Files Coverage Δ
planckton/utils/solvate.py 100.00% <ø> (ø)
planckton/__init__.py 100.00% <100.00%> (ø)
planckton/forcefields/__init__.py 100.00% <100.00%> (ø)
planckton/init.py 100.00% <100.00%> (ø)
planckton/sim.py 96.05% <100.00%> (+0.10%) :arrow_up:
planckton/utils/units.py 100.00% <100.00%> (ø)
jennyfothergill commented 3 years ago

I think I should switch this to https://pre-commit.com/ and https://github.com/pre-commit/action