LLNL / maestrowf

A tool to easily orchestrate general computational workflows both locally and on supercomputers
https://maestrowf.readthedocs.io
MIT License
134 stars 43 forks source link

Ci/test and release #396

Closed jwhite242 closed 2 years ago

jwhite242 commented 2 years ago

Turns on running of unit tests against multiple python versions upon pushes and pull requests

jwhite242 commented 2 years ago

So, general strategy here is to be a little greedy with the unit tests and not require linting to succeed first -> ideally catch errors in both in one go. Happy to change that though if nobody else likes that plan

FrankD412 commented 2 years ago

I'd prefer it fail linting -- I think it's a good incentive to have good style. We'll test your changes if you help us enforce style. Win-win in my book. 😝