ianholmeslab / bilby_encoder

BED-FASTA-BAM one-hot data encoder method utilizing state transition tuples.
0 stars 4 forks source link

Add GitHub Actions on PRs - linting, testing, etc. #12

Open ritster opened 6 days ago

ritster commented 6 days ago

Add testing and linting pipelines to verify PRs don't change intended behavior or diverge from code style guidance.

cmdcolin commented 6 days ago

is there an auto-formatter being used? black, or ruff or something like that? i can configure my editor to try to save using that format

ritster commented 6 days ago

I've just been using pylint as an arbitrary choice, but ruff seems to be very popular in practice. I'm happy with whichever option seems the best to everyone else!

cmdcolin commented 6 days ago

super. looks like things like ruff can be used in addition to pylint (not sure if pylint does formatting?), see "Advised linters alongside pylint" https://pypi.org/project/pylint/

ritster commented 6 days ago

I'm happy to just switch over to ruff - should be quicker, easier, more unified, etc.