choderalab / ensembler

Automated omics-scale protein modeling and simulation setup.
http://ensembler.readthedocs.io/
GNU General Public License v2.0
53 stars 21 forks source link

Updated README to add link to bioRxiv manuscript and Dryad database. #25

Closed jchodera closed 9 years ago

jchodera commented 9 years ago

Odd: https://travis-ci.org/choderalab/ensembler/builds/71865045#L322

Error: could not find environment: 2.7
danielparton commented 9 years ago

The problem with the Travis test here is that the Modeller license key, which I had stored using an encrypted environment variable, is not being picked up.

Apparently encrypted environment variables are not available to pull requests from forks due to the security risk of adding unknown code: http://docs.travis-ci.com/user/environment-variables/#Encrypted-Variables

danielparton commented 9 years ago

The Travis docs just recommend disabling tests which require secure environment variables: http://docs.travis-ci.com/user/pull-requests/#Security-Restrictions-when-testing-Pull-Requests

I could disable any Modeller-dependent tests for PRs. That seems like the simplest thing to do for now. Anyone else have advice here?

jchodera commented 9 years ago

I could disable any Modeller-dependent tests for PRs. That seems like the simplest thing to do for now.

Sounds reasonable! Go ahead and merge this and tackle that in a separate PR?

danielparton commented 9 years ago

K, will do. Thanks