search

joshspeagle / dynesty

Dynamic Nested Sampling package for computing Bayesian posteriors and evidences
https://dynesty.readthedocs.io/
MIT License
347 stars 76 forks source link

add pylint test to the test suite to catch trivial oversights #227

Closed segasai closed 3 years ago

segasai commented 3 years ago

To ensure the most obvious errors are caught, I've added pylint to the travis testing.

joshspeagle commented 3 years ago

Looks like this has some particular problem with CI that is causing the build the fail.

segasai commented 3 years ago

It's because I based on the branch without the last PR that you merged.

joshspeagle commented 3 years ago

Ah okay, will merge this in then.

joshspeagle commented 3 years ago

The pylint call looks like it breaks for Python 2.7 due to a missing config file (it works in Python 3.6+). Everything else about the code appears to work fine though.

segasai commented 3 years ago

To be honest 2.7 is EOL. Personally I'd retire it completely.